You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/03/13 12:09:10 UTC

[GitHub] [spark] DylanGuedes commented on issue #24071: [SPARK-27138][TESTS][KAFKA] Remove AdminUtils calls (fixes deprecation)

DylanGuedes commented on issue #24071: [SPARK-27138][TESTS][KAFKA] Remove AdminUtils calls (fixes deprecation)
URL: https://github.com/apache/spark/pull/24071#issuecomment-472395998
 
 
   @HeartSaVioR That makes sense, I'll start to investigate other deprecated calls (such as `ZkUtils`) then.
   
   Btw, is good to keep in mind that this way, reverting only this commit will replace only calls to AdminUtils, which can be desired in the future (i.e: maybe the Kafka team will only change one of the APIs, so you guys can atomically replace/revert only one of these calls).

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org