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.

Sign up for GitHub

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

Jump to bottom

Revert "Trying to add required flags to run druid using java 17 (#130)" #147

Merged
merged 1 commit into from
Jul 21, 2023
Merged

Revert "Trying to add required flags to run druid using java 17 (#130)" #147

merged 1 commit into from
Jul 21, 2023

Commits on Jul 20, 2023

  1. Revert "Trying to add required flags to run druid using java 17 (#130)" 

    This reverts our custom patch from commit 7cf2de4.

The necessary Java 17 exports are now included as part of 25.0.0
in https://github.com/confluentinc/druid/blob/25.0.0-confluent/examples/bin/run-java#L27-L56
which is now called by the druid.sh docker startup script as well.

The exports for java.base/jdk.internal.perf=ALL-UNNAMED are no longer
needed since apache#12481 (comment)
    @xvrl
    xvrl committed Jul 20, 2023
    Configuration menu
    Copy the full SHA
    8c5cc50 View commit details
    Browse the repository at this point in the history