HeartSaVioR edited a comment on issue #27191: [SPARK-30495][SS] Consider spark.security.credentials.kafka.enabled and cluster configuration when checking latest delegation token URL: https://github.com/apache/spark/pull/27191#issuecomment-574330429 >> If possible it's better to have another E2E test on Kafka security without delegation token. > The end to end test is really heavy weight and hard to maintain. Even after 2-3 fixes we've seen another failure most probably because of the environment. For me, it sounds as the test is really heavy weight and flaky because of the "environment", right? If they can be tested in same suite with setting up environment only once, it won't hurt. Doesn't mean to address in this PR, but E2E test would avoid introducing regression - this bug is a regression for the valid use case. I'll try to play with the test addition.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
