[GitHub] [spark] zuotingbing commented on issue #24569: [SPARK-23191][CORE] Warn rather than terminate when duplicate worker register happens

[GitBox <gi...@apache.org>]

zuotingbing commented on issue #24569: [SPARK-23191][CORE] Warn rather than terminate when duplicate worker register happens
URL: https://github.com/apache/spark/pull/24569#issuecomment-492489200
 
 
   > BTW in the case 2, I'm still a little confused.
   > 
   > > Master B receives register request again from the Worker
   > 
   > When does the first registration happen? in which step?
   
   master changed from vmax18 to vmax17.
   
   In master vmax18,  worker be removed because got no heartbeat but soon got heartbeat and asking to re-register with master vmax18(**will  tryRegisterAllMaster() which include master vmax17**).
   
   In the same time, worker has bean registered with master vmax17 when master vmax17 got leadership.
   
   So Worker registration failed: Duplicate worker ID with master vmax17 (register with master vmax17 twice ).

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