Re: [PR] [FLINK-33738][Scheduler] Make exponential-delay restart-strategy the default restart strategy [flink]

2024-01-09 Thread via GitHub
1996fanrui merged PR #24040: URL: https://github.com/apache/flink/pull/24040 -- 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. To unsubscribe, e-mail:

Re: [PR] [FLINK-33738][Scheduler] Make exponential-delay restart-strategy the default restart strategy [flink]

2024-01-09 Thread via GitHub
1996fanrui commented on PR #24040: URL: https://github.com/apache/flink/pull/24040#issuecomment-1884091150 Thanks to everyone for the review! The CI is green for now, merging~ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] [FLINK-33738][Scheduler] Make exponential-delay restart-strategy the default restart strategy [flink]

2024-01-09 Thread via GitHub
1996fanrui commented on code in PR #24040: URL: https://github.com/apache/flink/pull/24040#discussion_r1445953255 ## flink-core/src/main/java/org/apache/flink/configuration/RestartStrategyOptions.java: ## @@ -89,11 +89,10 @@ public class RestartStrategyOptions {

Re: [PR] [FLINK-33738][Scheduler] Make exponential-delay restart-strategy the default restart strategy [flink]

2024-01-09 Thread via GitHub
zhuzhurk commented on code in PR #24040: URL: https://github.com/apache/flink/pull/24040#discussion_r1445945589 ## flink-core/src/main/java/org/apache/flink/configuration/RestartStrategyOptions.java: ## @@ -89,11 +89,10 @@ public class RestartStrategyOptions {

Re: [PR] [FLINK-33738][Scheduler] Make exponential-delay restart-strategy the default restart strategy [flink]

2024-01-08 Thread via GitHub
qinf commented on PR #24040: URL: https://github.com/apache/flink/pull/24040#issuecomment-1882535964 Thank you @1996fanrui for your contribution, LGTM. -- 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

Re: [PR] [FLINK-33738][Scheduler] Make exponential-delay restart-strategy the default restart strategy [flink]

2024-01-07 Thread via GitHub
1996fanrui commented on code in PR #24040: URL: https://github.com/apache/flink/pull/24040#discussion_r1444232288 ## flink-core/src/main/java/org/apache/flink/configuration/RestartStrategyOptions.java: ## @@ -89,11 +89,10 @@ public class RestartStrategyOptions {

Re: [PR] [FLINK-33738][Scheduler] Make exponential-delay restart-strategy the default restart strategy [flink]

2024-01-07 Thread via GitHub
flinkbot commented on PR #24040: URL: https://github.com/apache/flink/pull/24040#issuecomment-1880359291 ## CI report: * 897dba2811c190461951040a2bd696fca4a18c88 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`