-
Notifications
You must be signed in to change notification settings - Fork 14k
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
KAFKA-17811: Separate modules to use different JDKs #17522
base: trunk
Are you sure you want to change the base?
Changes from 8 commits
1807f63
cd459fd
0b62146
8e739c5
47631b0
de5e1c2
694a598
35bc484
cd4697b
95fa392
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -47,7 +47,8 @@ plugins { | |
|
||
ext { | ||
gradleVersion = versions.gradle | ||
minJavaVersion = 11 | ||
minClientJavaVersion = 11 | ||
minServerJavaVersion = 17 | ||
buildVersionFileName = "kafka-version.properties" | ||
|
||
defaultMaxHeapSize = "2g" | ||
|
@@ -113,22 +114,27 @@ ext { | |
|
||
commitId = determineCommitId() | ||
|
||
configureJavaCompiler = { name, options -> | ||
isModuleNeedJava11 = { projectPath -> | ||
def modulesNeedingJava11 = [":clients", ":streams"] | ||
modulesNeedingJava11.any { projectPath.startsWith(it) } | ||
} | ||
|
||
configureJavaCompiler = { name, options, projectPath -> | ||
// -parameters generates arguments with parameter names in TestInfo#getDisplayName. | ||
// ref: https://github.com/junit-team/junit5/blob/4c0dddad1b96d4a20e92a2cd583954643ac56ac0/junit-jupiter-params/src/main/java/org/junit/jupiter/params/ParameterizedTest.java#L161-L164 | ||
if (name == "compileTestJava" || name == "compileTestScala") { | ||
|
||
def releaseVersion = isModuleNeedJava11(projectPath) ? minClientJavaVersion : minServerJavaVersion | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
|
||
options.compilerArgs << "-encoding" << "UTF-8" | ||
options.compilerArgs += ["--release", String.valueOf(releaseVersion)] | ||
|
||
if (name in ["compileTestJava", "compileTestScala"]) { | ||
options.compilerArgs << "-parameters" | ||
options.compilerArgs += ["--release", String.valueOf(minJavaVersion)] | ||
} else if (name == "compileJava" || name == "compileScala") { | ||
} else if (name in ["compileJava", "compileScala"]) { | ||
options.compilerArgs << "-Xlint:all" | ||
if (!project.path.startsWith(":connect") && !project.path.startsWith(":storage")) | ||
options.compilerArgs << "-Xlint:-rawtypes" | ||
options.compilerArgs << "-encoding" << "UTF-8" | ||
options.compilerArgs << "-Xlint:-rawtypes" | ||
options.compilerArgs << "-Xlint:-serial" | ||
options.compilerArgs << "-Xlint:-try" | ||
options.compilerArgs << "-Werror" | ||
options.compilerArgs += ["--release", String.valueOf(minJavaVersion)] | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could you please add |
||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. please update scala module as well |
||
} | ||
|
||
|
@@ -320,7 +326,7 @@ subprojects { | |
} | ||
|
||
tasks.withType(JavaCompile) { | ||
configureJavaCompiler(name, options) | ||
configureJavaCompiler(name, options, project.path) | ||
} | ||
|
||
if (shouldPublish) { | ||
|
@@ -726,7 +732,9 @@ subprojects { | |
} | ||
|
||
tasks.withType(ScalaCompile) { | ||
|
||
def modulesNeedingJava11 = [":clients", ":streams"] | ||
def releaseVersion = modulesNeedingJava11.any { project.path.startsWith(it) } ? | ||
minClientJavaVersion : minServerJavaVersion | ||
scalaCompileOptions.keepAliveMode = userKeepAliveMode | ||
|
||
scalaCompileOptions.additionalParameters = [ | ||
|
@@ -770,10 +778,9 @@ subprojects { | |
scalaCompileOptions.additionalParameters += ["-opt-warnings", "-Xlint:strict-unsealed-patmat"] | ||
// Scala 2.13.2 introduces compiler warnings suppression, which is a pre-requisite for -Xfatal-warnings | ||
scalaCompileOptions.additionalParameters += ["-Xfatal-warnings"] | ||
scalaCompileOptions.additionalParameters += ["--release", String.valueOf(releaseVersion)] | ||
|
||
scalaCompileOptions.additionalParameters += ["-release", String.valueOf(minJavaVersion)] | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
|
||
configureJavaCompiler(name, options) | ||
configureJavaCompiler(name, options, project.path) | ||
|
||
configure(scalaCompileOptions.forkOptions) { | ||
memoryMaximumSize = defaultMaxHeapSize | ||
|
@@ -2569,7 +2576,6 @@ project(':streams') { | |
// testCompileOnly prevents streams from exporting a dependency on test-utils, which would cause a dependency cycle | ||
testCompileOnly project(':streams:test-utils') | ||
|
||
testImplementation project(':metadata') | ||
testImplementation project(':clients').sourceSets.test.output | ||
testImplementation libs.reload4j | ||
testImplementation libs.junitJupiter | ||
|
@@ -2578,7 +2584,6 @@ project(':streams') { | |
testImplementation libs.mockitoCore | ||
testImplementation libs.mockitoJunitJupiter // supports MockitoExtension | ||
testImplementation libs.junitPlatformSuiteEngine // supports suite test | ||
testImplementation project(':group-coordinator') | ||
|
||
testRuntimeOnly project(':streams:test-utils') | ||
testRuntimeOnly runtimeTestLibs | ||
|
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.
Could you please move
modulesNeedingJava11
up to line#52?