tony-framework / TonY

TonY is a framework to natively run deep learning frameworks on Apache Hadoop.
https://tony-project.ai
Other
708 stars 164 forks source link

Make untrackedTaskFailed volatile #608

Closed zuston closed 3 years ago

oliverhu commented 3 years ago

@zuston can you add a bit more explanation in the commit message?

zuston commented 3 years ago

Sorry for lacking description. Bug introduced by PR #607 the param of untrackedTaskFailed will be used in two thread

  1. main thread to check the app status in monitor()method
  2. heartbeat thread

To keep param visibility between multiple thread, it's necessary to make untrackedTaskFailed volatile