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-6182] Disable conscrypt by default #7203
Conversation
cc: @chamikaramj (We may need to cherry pick this to the release branch.) |
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.
Please tag this PR with the associated JIRA.
A few minor comments, otherwise LGTM
"Dataflow runner uses conscrypt by default for SSL. To disable this feature, " | ||
+ "pass pipeline option --experiments={}", | ||
"Dataflow runner is using conscrypt SSL. To disable this feature, " | ||
+ "remove the pipeline option --experiments={}", | ||
experimentName); | ||
Security.insertProviderAt(new OpenSSLProvider(), 1); | ||
} else { | ||
LOG.info( |
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.
I would demote this to debug log. Since the Beam default is now consistent with Java default, it seems like it will be noise for most users.
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.
I would like to keep it as an info since this is a change in behavior for Dataflow runner.
...orker/src/main/java/org/apache/beam/runners/dataflow/worker/DataflowWorkerHarnessHelper.java
Show resolved
Hide resolved
...low-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/ExperimentContext.java
Show resolved
Hide resolved
Thank you @swegner. Addressed your questions. Will merge this after tests pass. |
* Disable conscrypt by default
Changing the default because conscrypt is continuing to cause stability issues. This change only affects Dataflow runner.
Follow this checklist to help us incorporate your contribution quickly and easily:
[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.It will help us expedite review of your Pull Request if you tag someone (e.g.
@username
) to look at it.Post-Commit Tests Status (on master branch)