Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CELEBORN-907][INFRA] The Jira Python misses our assignee when it searches users again #1832

Closed
wants to merge 1 commit into from

Conversation

yaooqinn
Copy link
Member

What changes were proposed in this pull request?

detailed desc can be found apache/spark@8fb799d

Why are the changes needed?

bypass upstream bug

Does this PR introduce any user-facing change?

no

How was this patch tested?

I guess @pan3793 has already hit the issue when resolving CELEBORN-903 at jira side

@codecov
Copy link

codecov bot commented Aug 24, 2023

Codecov Report

Merging #1832 (d241d86) into main (aa35c1c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1832   +/-   ##
=======================================
  Coverage   46.43%   46.43%           
=======================================
  Files         163      163           
  Lines       10135    10135           
  Branches      934      934           
=======================================
  Hits         4705     4705           
  Misses       5118     5118           
  Partials      312      312           

see 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@waitinfuture waitinfuture left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! Tested with #1829

@yaooqinn yaooqinn deleted the CELEBORN-907 branch August 24, 2023 03:55
@yaooqinn
Copy link
Member Author

thank you @waitinfuture

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants