[GitHub] [flink] dmvk commented on a diff in pull request #22603: [FLINK-32123][e2e] Fix E2E kafka download timeout

["dmvk (via GitHub)" <gi...@apache.org>]

dmvk commented on code in PR #22603:
URL: https://github.com/apache/flink/pull/22603#discussion_r1197426232


##########
flink-end-to-end-tests/test-scripts/kafka-common.sh:
##########
@@ -37,7 +37,7 @@ MAX_RETRY_SECONDS=120
 function setup_kafka_dist {
   # download Kafka
   mkdir -p $TEST_DATA_DIR
-  KAFKA_URL="https://archive.apache.org/dist/kafka/$KAFKA_VERSION/kafka_2.12-$KAFKA_VERSION.tgz"
+  KAFKA_URL="https://downloads.apache.org/kafka/$KAFKA_VERSION/kafka_2.12-$KAFKA_VERSION.tgz"

Review Comment:
   > I believe we are keeping up with Kafka upgrades anyway, as its a fundamental component.
   
   An upgrade must be deliberate on the Flink side; for example, if Kafka pushes a faulty release, we might want to skip it, but the older one would no longer be available.
   
   Also, we need to have some headroom for the upgrade without being stressed about all builds suddenly failing.



-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org