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/10/30 20:53:42 UTC

[GitHub] [spark] nishchalv opened a new pull request #26331: [SPARK-29670][core] Make executor's bindAddress configurable

nishchalv opened a new pull request #26331: [SPARK-29670][core] Make executor's bindAddress configurable
URL: https://github.com/apache/spark/pull/26331
 
 
   ### What changes were proposed in this pull request?
   With this change, executor's bindAddress is passed as an input parameter for RPCEnv.create.
   A previous PR https://github.com/apache/spark/pull/21261 which addressed the same, was using a Spark Conf property to get the bindAddress which wouldn't have worked for multiple executors.
   This PR is to enable anyone overriding CoarseGrainedExecutorBackend with their custom one to be able to invoke CoarseGrainedExecutorBackend.main() along with the option to configure bindAddress.
   
   ### Why are the changes needed?
   This is required when Kernel-based Virtual Machine (KVM)'s are used inside Linux container where the hostname is not the same as container hostname.
   
   ### Does this PR introduce any user-facing change?
   No
   
   ### How was this patch tested?
   Tested by running jobs with executors on KVMs inside a linux container.
   

----------------------------------------------------------------
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