-
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
[BEAM-10496] Eliminate some null errors from sdks/java/core #17819
Conversation
Run Java PreCommit |
0b1f3fd
to
1321418
Compare
Run Java PreCommit |
1321418
to
5423f4c
Compare
run java precommit |
Run Java_Examples_Dataflow_Java17 PreCommit |
amusingly, NPEs in elasticsearch test, plus flaking in pulsar test and file io |
Run Java PreCommit |
Our flakes are driving me nuts - confirmed some of these tests seem OK locally. I'll rebase and push just to get any recent fixes and have to kick jenkins less. |
5423f4c
to
8ce5a6f
Compare
Run Java PreCommit |
1 similar comment
Run Java PreCommit |
Run Java PreCommit |
@@ -99,16 +97,19 @@ public static Map<TupleTag<?>, PCollection<?>> fullyExpand( | |||
PCollection.class.getSimpleName(), | |||
valueComponent.getValue())); | |||
} | |||
@Nullable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How come the other ones doesn't need the @nullable?
@@ -180,7 +178,7 @@ public Class<? super T> getRawType() { | |||
|
|||
/** Returns the component type if this type is an array type, otherwise returns {@code null}. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this return null?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
@Nullable Object p1, | ||
@Nullable Object p2) { | ||
if (obj == null) { | ||
throw new IllegalStateException(lenientFormat(errorMessageTemplate, p1)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p2 not used
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
Any update on this PR? |
Still waiting to find time to resolve conflicts and address comments. |
8ce5a6f
to
c06fb49
Compare
c06fb49
to
1722ad9
Compare
Run Java PreCommit |
Hrmm. One of the licenses that we download the server is down. bouncycastle.org |
run java precommit |
Re-activated nullness type checking for a portion of
org.apache.beam.sdk.values
namespace.Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
R: @username
).[BEAM-XXX] Fixes bug in ApproximateQuantiles
, where you replaceBEAM-XXX
with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.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.