-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Adding support for PubSub Lite in Schema Transforms #24275
Conversation
ce913b5
to
c0ef59a
Compare
Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment |
Codecov Report
@@ Coverage Diff @@
## master #24275 +/- ##
==========================================
- Coverage 73.37% 73.36% -0.01%
==========================================
Files 719 718 -1
Lines 97206 97164 -42
==========================================
- Hits 71323 71286 -37
+ Misses 24536 24530 -6
- Partials 1347 1348 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Assigning reviewers. If you would like to opt out of this review, comment R: @robertwb for label java. Available commands:
The PR bot will only process comments in the main thread (not review comments). |
f10225e
to
f172553
Compare
LGTM from schema transform side of things |
f172553
to
208408a
Compare
Reminder, please take a look at this pr: @robertwb @Abacn @ahmedabu98 |
757569c
to
ea4114a
Compare
ea4114a
to
3fc6ab7
Compare
* Implement PubsubLiteReadSchemaTransform * fixup * fixup
* Implement PubsubLiteReadSchemaTransform * fixup * fixup
This PR depends on #24271
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
R: @username
).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, commentfixes #<ISSUE NUMBER>
instead.CHANGES.md
with noteworthy changes.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)
See CI.md for more information about GitHub Actions CI.