ijuma commented on code in PR #12948:
URL: https://github.com/apache/kafka/pull/12948#discussion_r1242312432
##########
Jenkinsfile:
##########
@@ -155,79 +155,23 @@ pipeline {
echo 'Skipping Kafka Streams archetype test for Java 17'
}
}
-
- // To avoid excessive Jenkins resource usage, we only run the stages
- // above at the PR stage. The ones below are executed after changes
- // are pushed to trunk and/or release branches. We achieve this via
- // the `when` clause.
-
- stage('JDK 8 and Scala 2.13') {
Review Comment:
That statement isn't strictly true since there could be bugs in the
implementation - that's why we had these tests in the first place. But the
truth is that they're low value since (1) the probability of a bug affecting
JDK 8 and Scala 2.13 but not JDK 8 and Scala 2.12 is low (2) having too many
variants makes it even less likely for people to pay attention to the flaky
failures that exist (3) it's expensive to run so many variants.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
