You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2022/06/10 03:30:43 UTC
[GitHub] [kafka] ijuma opened a new pull request, #12280: MINOR: Reuse gradle daemon for scala compilation (WIP)
ijuma opened a new pull request, #12280:
URL: https://github.com/apache/kafka/pull/12280
Draft PR testing the keep alive option for the gradle daemon when
compiling Scala code (only present in the gradle nightly atm).
After changing a single line in a test file in the streams-scala module,
the time to compile was reduced from 5 seconds to 1 second by
changing the keep alive option from `SESSION` to `DAEMON`.
`DAEMON` keep alive option:
> Task :streams:streams-scala:compileTestScala
> Watching 860 directories to track changes
> Caching disabled for task ':streams:streams-scala:compileTestScala' because:
> Build cache is disabled
> Task ':streams:streams-scala:compileTestScala' is not up-to-date because:
> Input property 'source' file /home/ijuma/src/kafka/streams/streams-scala/src/test/scala/org/apache/kafka/streams/scala/kstream/JoinedTest.scala has changed.
> Watching 852 directories to track changes
> Watching 851 directories to track changes
> Compiling with Zinc Scala compiler.
> Prepared Zinc Scala inputs: 0.005 secs
> compiling 16 Scala sources to /home/ijuma/src/kafka/streams/streams-scala/build/classes/scala/test ...
> done compiling
> Completed Scala compilation: 0.964 secs
> Watching 859 directories to track changes
> Watching 860 directories to track changes
> :streams:streams-scala:compileTestScala (Thread[Execution worker Thread 35,5,main]) completed. Took 0.991 secs.
`SESSION` keep alive option (the default):
> Task :streams:streams-scala:compileTestScala
> Watching 860 directories to track changes
> Caching disabled for task ‘:streams:streams-scala:compileTestScala’ because:
> Build cache is disabled
> Task ‘:streams:streams-scala:compileTestScala’ is not up-to-date because:
> Input property ‘source’ file /home/ijuma/src/kafka/streams/streams-scala/src/test/scala/org/apache/kafka/streams/scala/kstream/JoinedTest.scala has changed.
> Watching 852 directories to track changes
> Watching 851 directories to track changes
> Starting process ‘Gradle Worker Daemon 6’. Working directory: /home/ijuma/.gradle/workers Command: /usr/java/jdk-17.0.2/bin/java -Xss4m -XX:+UseParallelGC --add-opens=java.base/java.io=ALL-UNNAMED --add-opens=java.base/java.nio=ALL-UNNAMED --add-opens=java.base/java.nio.file=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.base/java.util.regex=ALL-UNNAMED --add-opens=java.base/java.util.stream=ALL-UNNAMED --add-opens=java.base/java.text=ALL-UNNAMED --add-opens=java.base/java.time=ALL-UNNAMED --add-opens=java.security.jgss/sun.security.krb5=ALL-UNNAMED @/home/ijuma/.gradle/.tmp/gradle-worker-classpath1215126969431063029txt -Xmx2048m -Dfile.encoding=UTF-8 -Duser.country=US -Duser.language=en -Duser.variant [worker.org](http://worker.org/).gradle.process.internal.worker.GradleWorkerMain ‘Gradle Worker Daemon 6’
> Successfully started process ‘Gradle Worker Daemon 6’
> Started Gradle worker daemon (0.227 secs) with fork options DaemonForkOptions{executable=/usr/java/jdk-17.0.2/bin/java, minHeapSize=null, maxHeapSize=2048m, jvmArgs=[-Xss4m, -XX:+UseParallelGC, --add-opens=java.base/java.io=ALL-UNNAMED, --add-opens=java.base/java.nio=ALL-UNNAMED, --add-opens=java.base/java.nio.file=ALL-UNNAMED, --add-opens=java.base/java.util.concurrent=ALL-UNNAMED, --add-opens=java.base/java.util.regex=ALL-UNNAMED, --add-opens=java.base/java.util.stream=ALL-UNNAMED, --add-opens=java.base/java.text=ALL-UNNAMED, --add-opens=java.base/java.time=ALL-UNNAMED, --add-opens=java.security.jgss/sun.security.krb5=ALL-UNNAMED], keepAliveMode=SESSION}.
> Compiling with Zinc Scala compiler.
> Prepared Zinc Scala inputs: 0.183 secs
> compiling 16 Scala sources to /home/ijuma/src/kafka/streams/streams-scala/build/classes/scala/test ...
> done compiling
> Completed Scala compilation: 5.256 secs
> Watching 859 directories to track changes
> Watching 860 directories to track changes
> :streams:streams-scala:compileTestScala (Thread[Execution worker Thread 29,5,main]) completed. Took 5.79 secs.
>
### Committer Checklist (excluded from commit message)
- [ ] Verify design and implementation
- [ ] Verify test coverage and CI build status
- [ ] Verify documentation (including upgrade notes)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] ijuma merged pull request #12280: MINOR: Reuse gradle daemon for scala compilation by default
Posted by GitBox <gi...@apache.org>.
ijuma merged PR #12280:
URL: https://github.com/apache/kafka/pull/12280
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [kafka] ijuma commented on pull request #12280: MINOR: Reuse gradle daemon for scala compilation by default
Posted by GitBox <gi...@apache.org>.
ijuma commented on PR #12280:
URL: https://github.com/apache/kafka/pull/12280#issuecomment-1333777333
There is quite a lot of unit test flakiness, but it doesn't seem related to this PR (it's happening in trunk too).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org