Re: [PR] [FLINK-33441] Implement restore tests for ExecUnion node [flink]

2023-12-12 Thread via GitHub
dawidwys merged PR #23653: URL: https://github.com/apache/flink/pull/23653 -- 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: issues-unsubscr...@flink.apac

Re: [PR] [FLINK-33441] Implement restore tests for ExecUnion node [flink]

2023-12-08 Thread via GitHub
jnh5y commented on code in PR #23653: URL: https://github.com/apache/flink/pull/23653#discussion_r1420934618 ## flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/plan/nodes/exec/testutils/UnionTestPrograms.java: ## @@ -0,0 +1,158 @@ +/* + * Licensed to

Re: [PR] [FLINK-33441] Implement restore tests for ExecUnion node [flink]

2023-12-08 Thread via GitHub
bvarghese1 commented on code in PR #23653: URL: https://github.com/apache/flink/pull/23653#discussion_r1420874894 ## flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/plan/nodes/exec/testutils/RestoreTestBase.java: ## @@ -91,7 +91,8 @@ public EnumSet s

Re: [PR] [FLINK-33441] Implement restore tests for ExecUnion node [flink]

2023-12-05 Thread via GitHub
jnh5y commented on code in PR #23653: URL: https://github.com/apache/flink/pull/23653#discussion_r1415705636 ## flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/plan/nodes/exec/testutils/RestoreTestBase.java: ## @@ -91,7 +91,8 @@ public EnumSet suppor

Re: [PR] [FLINK-33441] Implement restore tests for ExecUnion node [flink]

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

[PR] [FLINK-33441] Implement restore tests for ExecUnion node [flink]

2023-11-02 Thread via GitHub
bvarghese1 opened a new pull request, #23653: URL: https://github.com/apache/flink/pull/23653 ## What is the purpose of the change Add restore tests for ExecUnion node ## Verifying this change This change added tests and can be verified as follows: -