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

Cherry picking PR #28618 into 2.51.0 (setting numShards for Python BigQuery xlang) #28631

Merged
merged 8 commits into from
Sep 27, 2023

Conversation

ahmedabu98
Copy link
Contributor

Cherry picking PR #28618 into 2.51.0

@ahmedabu98
Copy link
Contributor Author

R: @kennknowles

@github-actions
Copy link
Contributor

Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control

@ahmedabu98
Copy link
Contributor Author

Run Python_Xlang_Gcp_Dataflow PostCommit

@ahmedabu98 ahmedabu98 changed the base branch from master to release-2.51.0 September 23, 2023 00:33
@ahmedabu98
Copy link
Contributor Author

Run Python_Xlang_Gcp_Direct PostCommit

@ahmedabu98
Copy link
Contributor Author

Previous iteration of xlang direct GCP tests failed because it was running on another PR concurrently.. shows another occurrence of #27445

@Abacn
Copy link
Contributor

Abacn commented Sep 23, 2023

fyi test passed on GitHub Action also: https://github.com/apache/beam/actions/workflows/beam_PostCommit_Python_Xlang_Gcp_Direct.yml?query=event%3Aissue_comment

Though current phrase trigger on GitHub Action is not conveniently visible (we're investigating a solution)

@codecov
Copy link

codecov bot commented Sep 23, 2023

Codecov Report

Merging #28631 (490605b) into release-2.51.0 (6306b21) will increase coverage by 0.00%.
The diff coverage is 50.00%.

@@               Coverage Diff               @@
##           release-2.51.0   #28631   +/-   ##
===============================================
  Coverage           72.23%   72.23%           
===============================================
  Files                 684      684           
  Lines              101071   101073    +2     
===============================================
+ Hits                73004    73009    +5     
+ Misses              26491    26488    -3     
  Partials             1576     1576           
Flag Coverage Δ
python 82.81% <50.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
sdks/python/apache_beam/io/gcp/bigquery.py 69.92% <50.00%> (-0.05%) ⬇️

... and 4 files with indirect coverage changes

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

@ahmedabu98 ahmedabu98 changed the title Cherry picking PR #28618 into 2.51.0 Cherry picking PR #28618 into 2.51.0 (setting numShards for Python BigQuery xlang) Sep 23, 2023
@ahmedabu98
Copy link
Contributor Author

Run Python_Xlang_Gcp_Dataflow PostCommit

@kennknowles
Copy link
Member

PreCommit Python Dataframes / Test Results (pull_request_target) only has the option to re-run all jobs. I don't know what causes that but I'm afraid of causing flakiness by rerunning too much. WDYT? Is that the way to go? Is there a real problem? It didn't dump me to the console I was expecting to see in order to triage it.

@kennknowles
Copy link
Member

Ah, I see the failures are still Fhir. Why is it running in the dataframes precommit? Well anyhow I will merge this cherrypick also. However, those jobs should be disabled until they are healthy.

@kennknowles kennknowles merged commit 2420c90 into apache:release-2.51.0 Sep 27, 2023
95 of 98 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants