[GitHub] johnyangk commented on issue #21: [NEMO-46] Make the operations on ExecutorRegistry atomic

2018-05-23 Thread GitBox
johnyangk commented on issue #21: [NEMO-46] Make the operations on ExecutorRegistry atomic URL: https://github.com/apache/incubator-nemo/pull/21#issuecomment-391586301 @bgchun We have some good tests, but they can be improved. The `scheduler` package is covered relatively well. We also hav

[GitHub] johnyangk commented on issue #21: [NEMO-46] Make the operations on ExecutorRegistry atomic

2018-05-23 Thread GitBox
johnyangk commented on issue #21: [NEMO-46] Make the operations on ExecutorRegistry atomic URL: https://github.com/apache/incubator-nemo/pull/21#issuecomment-391575534 @jeongyooneo Good catch. I've update the branch to reflect your comments.

[GitHub] johnyangk commented on issue #21: [NEMO-46] Make the operations on ExecutorRegistry atomic

2018-05-23 Thread GitBox
johnyangk commented on issue #21: [NEMO-46] Make the operations on ExecutorRegistry atomic URL: https://github.com/apache/incubator-nemo/pull/21#issuecomment-391552787 @jeongyooneo Thanks! I've addressed your comments. This i