Re: [PR] [BP-1.18][FLINK-33276][ci] Merges connect_1 and connect_2 stages into a single one [flink]

2023-11-15 Thread via GitHub
XComp merged PR #23719: URL: https://github.com/apache/flink/pull/23719 -- 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] [BP-1.18][FLINK-33276][ci] Merges connect_1 and connect_2 stages into a single one [flink]

2023-11-15 Thread via GitHub
XComp commented on PR #23719: URL: https://github.com/apache/flink/pull/23719#issuecomment-1812507090 Yes, that change was rather meant to reduce the amount of runners that I need to use in the GHA test. And I thought that it would be valuable to put it into `master` as well.

Re: [PR] [BP-1.18][FLINK-33276][ci] Merges connect_1 and connect_2 stages into a single one [flink]

2023-11-15 Thread via GitHub
zentol commented on PR #23719: URL: https://github.com/apache/flink/pull/23719#issuecomment-1812448493 Admittedly that could be a follow-up. 樂 -- 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

Re: [PR] [BP-1.18][FLINK-33276][ci] Merges connect_1 and connect_2 stages into a single one [flink]

2023-11-15 Thread via GitHub
zentol commented on PR #23719: URL: https://github.com/apache/flink/pull/23719#issuecomment-1812448008 couldn't we have used the freed runner slot to parallelise something else? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] [BP-1.18][FLINK-33276][ci] Merges connect_1 and connect_2 stages into a single one [flink]

2023-11-15 Thread via GitHub
flinkbot commented on PR #23719: URL: https://github.com/apache/flink/pull/23719#issuecomment-1812425572 ## CI report: * cbb25a4fee81c6aaaed7ba666f8be65570cf32c5 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`