[GitHub] [flink] liuyongvs commented on pull request #22267: [FLINK-31602][table] Add built-in ARRAY_POSITION function.

2023-03-30 Thread via GitHub
liuyongvs commented on PR #22267: URL: https://github.com/apache/flink/pull/22267#issuecomment-1489960245 fix the description, thanks for your review @luoyuxia -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [flink] liuyongvs commented on pull request #22267: [FLINK-31602][table] Add built-in ARRAY_POSITION function.

2023-03-29 Thread via GitHub
liuyongvs commented on PR #22267: URL: https://github.com/apache/flink/pull/22267#issuecomment-1489627172 @luoyuxia fix your review and rebase master to verify whether it is affected by others. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [flink] liuyongvs commented on pull request #22267: [FLINK-31602][table] Add built-in ARRAY_POSITION function.

2023-03-27 Thread via GitHub
liuyongvs commented on PR #22267: URL: https://github.com/apache/flink/pull/22267#issuecomment-1484975513 @damumu0625 fixed the description of spark links -- 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 t

[GitHub] [flink] liuyongvs commented on pull request #22267: [FLINK-31602][table] Add built-in ARRAY_POSITION function.

2023-03-26 Thread via GitHub
liuyongvs commented on PR #22267: URL: https://github.com/apache/flink/pull/22267#issuecomment-1484393251 hi @snuyanzin do you have time to review it again? thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [flink] liuyongvs commented on pull request #22267: [FLINK-31602][table] Add built-in ARRAY_POSITION function.

2023-03-26 Thread via GitHub
liuyongvs commented on PR #22267: URL: https://github.com/apache/flink/pull/22267#issuecomment-1484392537 rebase to fix conflict with array_remove which merged just now -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t