Skip to content

[#22181] Fix java package for SDK java core benchmark org.apache.beam.sdk.schemas -> org.apache.beam.sdk.jmh.schemas - #22346

Merged
lukecwik merged 2 commits into
apache:masterfrom
lukecwik:22181
Jul 20, 2022
Merged

[#22181] Fix java package for SDK java core benchmark org.apache.beam.sdk.schemas -> org.apache.beam.sdk.jmh.schemas#22346
lukecwik merged 2 commits into
apache:masterfrom
lukecwik:22181

Conversation

@lukecwik

Copy link
Copy Markdown
Member

Missed during review of #22182


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Choose reviewer(s) and mention them in a comment (R: @username).
  • Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
  • Update CHANGES.md with noteworthy changes.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md

GitHub Actions Tests Status (on master branch)

Build python source distribution and wheels
Python tests
Java tests

See CI.md for more information about GitHub Actions CI.

…e.beam.sdk.schemas -> org.apache.beam.sdk.jmh.schemas
Missed during review of apache#22182
@lukecwik

Copy link
Copy Markdown
MemberAuthor

R: @mosche@TheNeuralBit

@github-actions

Copy link
Copy Markdown
Contributor

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

@lukecwik

Copy link
Copy Markdown
MemberAuthor

Run GoPortable PreCommit

@lukecwik

Copy link
Copy Markdown
MemberAuthor

Run Java_PVR_Flink_Docker PreCommit

@codecov

codecovBot commented Jul 19, 2022

Copy link
Copy Markdown

Codecov Report

Merging #22346 (203cb13) into master (abcc9e0) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@ Coverage Diff @@## master #22346 +/- ##
==========================================
- Coverage 74.17% 74.16% -0.01% 
==========================================
Files 706 706 Lines 93190 93190 ==========================================
- Hits 69121 69113 -8 - Misses 22801 22809 +8 
Partials 1268 1268 
FlagCoverage Δ
python83.53% <ø> (-0.02%)⬇️

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

Impacted FilesCoverage Δ
.../python/apache_beam/testing/test_stream_service.py88.09% <0.00%> (-4.77%)⬇️
...che_beam/runners/interactive/interactive_runner.py90.06% <0.00%> (-1.33%)⬇️
...eam/runners/portability/fn_api_runner/execution.py92.44% <0.00%> (-0.65%)⬇️
sdks/python/apache_beam/transforms/combiners.py93.05% <0.00%> (-0.39%)⬇️
...hon/apache_beam/runners/worker/bundle_processor.py93.54% <0.00%> (-0.13%)⬇️
...ks/python/apache_beam/runners/worker/sdk_worker.py89.09% <0.00%> (+0.15%)⬆️
...eam/runners/interactive/interactive_environment.py92.02% <0.00%> (+0.30%)⬆️
sdks/python/apache_beam/utils/interactive_utils.py97.56% <0.00%> (+2.43%)⬆️

@lukecwik

Copy link
Copy Markdown
MemberAuthor

Run GoPortable PreCommit

@lukecwik

Copy link
Copy Markdown
MemberAuthor

Run Java PreCommit

@lukecwik

Copy link
Copy Markdown
MemberAuthor

Run Python PreCommit

@lukecwik

Copy link
Copy Markdown
MemberAuthor

Run Python_PVR_Flink PreCommit

@lukecwik

Copy link
Copy Markdown
MemberAuthor

Run Python_PVR_Flink_Docker PreCommit

@lukecwik

Copy link
Copy Markdown
MemberAuthor

Run Java_PVR_Flink_Docker PreCommit

@lukecwik
lukecwik merged commit 9352093 into apache:masterJul 20, 2022
@mosche

Copy link
Copy Markdown
Member

Thanks for catching that @lukecwik

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@lukecwik@mosche@aaltay