Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Fix paiToken update logic #976

Merged
merged 71 commits into from
Apr 17, 2019
Merged

Conversation

SparkSnail
Copy link
Contributor

@SparkSnail SparkSnail commented Apr 11, 2019

When update paiToken after experiment is started, do not throw error.

@SparkSnail SparkSnail changed the title Fix paiToken timeout error Fix paiToken update logic Apr 11, 2019
@@ -529,7 +529,14 @@ class PAITrainingService implements TrainingService {

private async statusCheckingLoop(): Promise<void> {
while (!this.stopping) {
await this.updatePaiToken();
try{
await this.updatePaiToken();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if timeout, there is no log written.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if timeout, there is no log written.

when catch error, log it out here.

@SparkSnail SparkSnail merged commit 9a3c61e into microsoft:master Apr 17, 2019
@SparkSnail SparkSnail deleted the fix-pai-token branch April 18, 2019 06:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants