[GitHub] [flink] wuchong commented on a diff in pull request #22734: [FLINK-32277][table-runtime] Introduce the basic operator fusion codegen framework

2023-06-30 Thread via GitHub
wuchong commented on code in PR #22734: URL: https://github.com/apache/flink/pull/22734#discussion_r1248518654 ## flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/planner/plan/fusion/spec/HashJoinFusionCodegenSpec.scala: ## @@ -0,0 +1,542 @@ +/* + *

[GitHub] [flink] wuchong commented on a diff in pull request #22734: [FLINK-32277][table-runtime] Introduce the basic operator fusion codegen framework

2023-06-30 Thread via GitHub
wuchong commented on code in PR #22734: URL: https://github.com/apache/flink/pull/22734#discussion_r1248518300 ## flink-table/flink-table-runtime/src/test/java/org/apache/flink/table/runtime/operators/multipleinput/input/InputSelectionHandlerTest.java: ## @@ -41,7 +42,11 @@

[GitHub] [flink] wuchong commented on a diff in pull request #22734: [FLINK-32277][table-runtime] Introduce the basic operator fusion codegen framework

2023-06-30 Thread via GitHub
wuchong commented on code in PR #22734: URL: https://github.com/apache/flink/pull/22734#discussion_r1248506769 ## flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/planner/codegen/GeneratedExpression.scala: ## @@ -39,7 +39,7 @@ import

[GitHub] [flink] liuyongvs commented on a diff in pull request #22745: [FLINK-31691][table] Add built-in MAP_FROM_ENTRIES function.

2023-06-30 Thread via GitHub
liuyongvs commented on code in PR #22745: URL: https://github.com/apache/flink/pull/22745#discussion_r1248456341 ## docs/data/sql_functions.yml: ## @@ -658,6 +658,9 @@ collection: - sql: MAP_FROM_ARRAYS(array_of_keys, array_of_values) table: mapFromArrays(array_of_keys,

[GitHub] [flink] liuyongvs commented on pull request #22745: [FLINK-31691][table] Add built-in MAP_FROM_ENTRIES function.

2023-06-30 Thread via GitHub
liuyongvs commented on PR #22745: URL: https://github.com/apache/flink/pull/22745#issuecomment-1615454677 Hi @snuyanzin do you have time to review it again -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[jira] [Commented] (FLINK-32418) ClassNotFoundException when using flink-protobuf with sql-client

2023-06-30 Thread Benchao Li (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32418?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739266#comment-17739266 ] Benchao Li commented on FLINK-32418: Agree with [~yuxia] , connectors and formats should use the

[GitHub] [flink] hanyuzheng7 commented on pull request #22923: [FLINK-32296][table] Support cast of collections with row element types

2023-06-30 Thread via GitHub
hanyuzheng7 commented on PR #22923: URL: https://github.com/apache/flink/pull/22923#issuecomment-1615417080 @snuyanzin do you know the result format of this select statement in sql-client? `SELECT cast(array[row(1, 'one'), row(2, 'two')] as array>);` -- This is an automated message

[GitHub] [flink] flinkbot commented on pull request #22924: [FLINK-32404][table] Add catalog modification listener interface and create listener for catalog manager

2023-06-30 Thread via GitHub
flinkbot commented on PR #22924: URL: https://github.com/apache/flink/pull/22924#issuecomment-1615407096 ## CI report: * b0fc06c58850968ad345c0b2693a4387a648cc9d UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[jira] [Updated] (FLINK-32404) Introduce catalog modification listener and factory interfaces

2023-06-30 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32404?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated FLINK-32404: --- Labels: pull-request-available (was: ) > Introduce catalog modification listener and

[GitHub] [flink] FangYongs opened a new pull request, #22924: [FLINK-32404][table] Add catalog modification listener interface and create listener for catalog manager

2023-06-30 Thread via GitHub
FangYongs opened a new pull request, #22924: URL: https://github.com/apache/flink/pull/22924 ## What is the purpose of the change This PR aims to add catalog modification listener interface and create the listeners for catalog manager from configuration ## Brief change log

[GitHub] [flink] xintongsong commented on a diff in pull request #22851: [FLINK-31646][network] Implement the remote tier producer for the tiered storage

2023-06-30 Thread via GitHub
xintongsong commented on code in PR #22851: URL: https://github.com/apache/flink/pull/22851#discussion_r1248387674 ## flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/hybrid/tiered/file/HashPartitionFileWriter.java: ## @@ -0,0 +1,234 @@ +/* + * Licensed

[GitHub] [flink] FangYongs commented on a diff in pull request #22882: [FLINK-32370][runtime] Revert print error log in AbstractHandler and skip check exceptions in jdbc driver e2e tests which will ch

2023-06-30 Thread via GitHub
FangYongs commented on code in PR #22882: URL: https://github.com/apache/flink/pull/22882#discussion_r1248421262 ## flink-end-to-end-tests/run-nightly-tests.sh: ## @@ -185,7 +185,7 @@ function run_group_2 { run_test "Batch SQL end-to-end test using hybrid selective

[GitHub] [flink] FangYongs commented on a diff in pull request #22882: [FLINK-32370][runtime] Revert print error log in AbstractHandler and skip check exceptions in jdbc driver e2e tests which will ch

2023-06-30 Thread via GitHub
FangYongs commented on code in PR #22882: URL: https://github.com/apache/flink/pull/22882#discussion_r1248421262 ## flink-end-to-end-tests/run-nightly-tests.sh: ## @@ -185,7 +185,7 @@ function run_group_2 { run_test "Batch SQL end-to-end test using hybrid selective

[GitHub] [flink] FangYongs commented on a diff in pull request #22882: [FLINK-32370][runtime] Revert print error log in AbstractHandler and skip check exceptions in jdbc driver e2e tests which will ch

2023-06-30 Thread via GitHub
FangYongs commented on code in PR #22882: URL: https://github.com/apache/flink/pull/22882#discussion_r1248422378 ## flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/AbstractHandler.java: ## @@ -268,12 +266,7 @@ private CompletableFuture handleException(

[GitHub] [flink] FangYongs commented on a diff in pull request #22882: [FLINK-32370][runtime] Revert print error log in AbstractHandler and skip check exceptions in jdbc driver e2e tests which will ch

2023-06-30 Thread via GitHub
FangYongs commented on code in PR #22882: URL: https://github.com/apache/flink/pull/22882#discussion_r1248421262 ## flink-end-to-end-tests/run-nightly-tests.sh: ## @@ -185,7 +185,7 @@ function run_group_2 { run_test "Batch SQL end-to-end test using hybrid selective

[GitHub] [flink] flinkbot commented on pull request #22923: [FLINK-32296][table] Support cast of collections with row element types

2023-06-30 Thread via GitHub
flinkbot commented on PR #22923: URL: https://github.com/apache/flink/pull/22923#issuecomment-1615306688 ## CI report: * 141305019b2b1a9eadc73be6c4e3bf7662460a0e UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[jira] [Assigned] (FLINK-32296) Flink SQL handle array of row incorrectly

2023-06-30 Thread Sergey Nuyanzin (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32296?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sergey Nuyanzin reassigned FLINK-32296: --- Assignee: Sergey Nuyanzin > Flink SQL handle array of row incorrectly >

[jira] [Commented] (FLINK-32296) Flink SQL handle array of row incorrectly

2023-06-30 Thread Sergey Nuyanzin (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739253#comment-17739253 ] Sergey Nuyanzin commented on FLINK-32296: - [~qingwei91] could you please double check that this

[GitHub] [flink] snuyanzin opened a new pull request, #22923: [FLINK-32296][table] Support cast of collections with row element types

2023-06-30 Thread via GitHub
snuyanzin opened a new pull request, #22923: URL: https://github.com/apache/flink/pull/22923 The PR addresses allows casting in correct way collections like `ARRAY` and `MAP` with `ROW` element types e.g. ```sql SELECT cast(array[row(1, 'one'), row(2, 'two')] as array>); SELECT

[jira] [Updated] (FLINK-32296) Flink SQL handle array of row incorrectly

2023-06-30 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32296?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated FLINK-32296: --- Labels: pull-request-available (was: ) > Flink SQL handle array of row incorrectly >

[jira] [Comment Edited] (FLINK-32296) Flink SQL handle array of row incorrectly

2023-06-30 Thread Sergey Nuyanzin (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739252#comment-17739252 ] Sergey Nuyanzin edited comment on FLINK-32296 at 7/1/23 12:14 AM: -- The

[jira] [Comment Edited] (FLINK-32296) Flink SQL handle array of row incorrectly

2023-06-30 Thread Sergey Nuyanzin (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739252#comment-17739252 ] Sergey Nuyanzin edited comment on FLINK-32296 at 7/1/23 12:14 AM: -- The

[jira] [Commented] (FLINK-32296) Flink SQL handle array of row incorrectly

2023-06-30 Thread Sergey Nuyanzin (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739252#comment-17739252 ] Sergey Nuyanzin commented on FLINK-32296: - The root cause is {{RowToRowCastRule}}. Since it was

[jira] [Commented] (FLINK-32505) Compilation error in ProducerMergedPartitionFileWriter

2023-06-30 Thread Yuxin Tan (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32505?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739251#comment-17739251 ] Yuxin Tan commented on FLINK-32505: --- [~mapohl] Thanks for the fix. > Compilation error in

[GitHub] [flink] liuyongvs commented on a diff in pull request #22917: [FLINK-32498][table] Fix the problem of array_max return type, it sho…

2023-06-30 Thread via GitHub
liuyongvs commented on code in PR #22917: URL: https://github.com/apache/flink/pull/22917#discussion_r1248327617 ## flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/inference/strategies/SpecificTypeStrategies.java: ## @@ -58,8 +58,12 @@ public final

[GitHub] [flink] hanyuzheng7 commented on a diff in pull request #22917: [FLINK-32498][table] Fix the problem of array_max return type, it sho…

2023-06-30 Thread via GitHub
hanyuzheng7 commented on code in PR #22917: URL: https://github.com/apache/flink/pull/22917#discussion_r1248318340 ## flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/inference/strategies/SpecificTypeStrategies.java: ## @@ -58,8 +58,12 @@ public final

[GitHub] [flink] hanyuzheng7 commented on a diff in pull request #22917: [FLINK-32498][table] Fix the problem of array_max return type, it sho…

2023-06-30 Thread via GitHub
hanyuzheng7 commented on code in PR #22917: URL: https://github.com/apache/flink/pull/22917#discussion_r1248317688 ## flink-table/flink-table-common/src/main/java/org/apache/flink/table/functions/BuiltInFunctionDefinitions.java: ## @@ -303,7 +302,7 @@ ANY,

[GitHub] [flink] hanyuzheng7 commented on a diff in pull request #22917: [FLINK-32498][table] Fix the problem of array_max return type, it sho…

2023-06-30 Thread via GitHub
hanyuzheng7 commented on code in PR #22917: URL: https://github.com/apache/flink/pull/22917#discussion_r1248317688 ## flink-table/flink-table-common/src/main/java/org/apache/flink/table/functions/BuiltInFunctionDefinitions.java: ## @@ -303,7 +302,7 @@ ANY,

[GitHub] [flink] hanyuzheng7 commented on pull request #22922: [FLINK-32256][table] Add built-in ARRAY_ MIN function.

2023-06-30 Thread via GitHub
hanyuzheng7 commented on PR #22922: URL: https://github.com/apache/flink/pull/22922#issuecomment-1615226953 @flinkbot run azure -- 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

[GitHub] [flink] XComp commented on pull request #22661: [FLINK-31783][runtime] Migrates DefaultLeaderElectionService from LeaderElectionDriver to the MultipleComponentLeaderElectionDriver interface

2023-06-30 Thread via GitHub
XComp commented on PR #22661: URL: https://github.com/apache/flink/pull/22661#issuecomment-1615206613 @flinkbot run azure -- 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.

[GitHub] [flink] XComp commented on pull request #22661: [FLINK-31783][runtime] Migrates DefaultLeaderElectionService from LeaderElectionDriver to the MultipleComponentLeaderElectionDriver interface

2023-06-30 Thread via GitHub
XComp commented on PR #22661: URL: https://github.com/apache/flink/pull/22661#issuecomment-1615206433 Test instability is related to FLINK-32370 -- 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

[GitHub] [flink] flinkbot commented on pull request #22922: [FLINK-32256][table] Add built-in ARRAY_ MIN function.

2023-06-30 Thread via GitHub
flinkbot commented on PR #22922: URL: https://github.com/apache/flink/pull/22922#issuecomment-1615182489 ## CI report: * 86adecb4b4efea7984e8d04a5194c2d6b2defb47 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[jira] [Updated] (FLINK-32256) Add ARRAY_MIN support in SQL & Table API

2023-06-30 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32256?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated FLINK-32256: --- Labels: pull-request-available (was: ) > Add ARRAY_MIN support in SQL & Table API >

[GitHub] [flink] hanyuzheng7 opened a new pull request, #22922: [FLINK-32256][table] Add built-in ARRAY_ MIN function.

2023-06-30 Thread via GitHub
hanyuzheng7 opened a new pull request, #22922: URL: https://github.com/apache/flink/pull/22922 ## What is the purpose of the change *(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring

[jira] [Commented] (FLINK-32257) Add ARRAY_MAX support in SQL & Table API

2023-06-30 Thread Hanyu Zheng (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739229#comment-17739229 ] Hanyu Zheng commented on FLINK-32257: - Hi [~jackylau], I am also working on collection functions. I

[GitHub] [flink] hanyuzheng7 commented on pull request #22785: [FLINK-32259][table] Add built-in ARRAY_JOIN function.

2023-06-30 Thread via GitHub
hanyuzheng7 commented on PR #22785: URL: https://github.com/apache/flink/pull/22785#issuecomment-1615158354 @dawidwys I have already rebased it and solved conflicts and run CI test. It's ready to merge now. Thank you! -- This is an automated message from the Apache Git Service. To

[jira] [Created] (FLINK-32508) Flink-Metrics Prometheus - Native Histograms / Native Counters

2023-06-30 Thread Ryan van Huuksloot (Jira)
Ryan van Huuksloot created FLINK-32508: -- Summary: Flink-Metrics Prometheus - Native Histograms / Native Counters Key: FLINK-32508 URL: https://issues.apache.org/jira/browse/FLINK-32508 Project:

[jira] [Updated] (FLINK-32507) Document KafkaSink SinkWriterMetricGroup metrics

2023-06-30 Thread Mason Chen (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32507?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mason Chen updated FLINK-32507: --- Issue Type: Improvement (was: Technical Debt) > Document KafkaSink SinkWriterMetricGroup metrics >

[jira] [Resolved] (FLINK-31662) Migrate flink-connectors-aws to new structure

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-31662?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Danny Cranmer resolved FLINK-31662. --- Resolution: Done > Migrate flink-connectors-aws to new structure >

[jira] [Commented] (FLINK-31662) Migrate flink-connectors-aws to new structure

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-31662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739210#comment-17739210 ] Danny Cranmer commented on FLINK-31662: --- Merged commit

[jira] [Resolved] (FLINK-30946) Migrate AWS connector formats to flink-connector-aws

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-30946?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Danny Cranmer resolved FLINK-30946. --- Assignee: Hong Liang Teoh Resolution: Fixed > Migrate AWS connector formats to

[jira] [Updated] (FLINK-31662) Migrate flink-connectors-aws to new structure

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-31662?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Danny Cranmer updated FLINK-31662: -- Component/s: Connectors / DynamoDB Connectors / Firehose

[jira] [Updated] (FLINK-30946) Migrate AWS connector formats to flink-connector-aws

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-30946?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Danny Cranmer updated FLINK-30946: -- Fix Version/s: aws-connector-4.2.0 > Migrate AWS connector formats to flink-connector-aws >

[jira] [Commented] (FLINK-30946) Migrate AWS connector formats to flink-connector-aws

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-30946?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739211#comment-17739211 ] Danny Cranmer commented on FLINK-30946: --- All subtasks complete. Resolving, thanks [~liangtl]  >

[jira] [Updated] (FLINK-31662) Migrate flink-connectors-aws to new structure

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-31662?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Danny Cranmer updated FLINK-31662: -- Component/s: Connectors / AWS > Migrate flink-connectors-aws to new structure >

[jira] [Updated] (FLINK-31662) Migrate flink-connectors-aws to new structure

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-31662?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Danny Cranmer updated FLINK-31662: -- Fix Version/s: aws-connector-4.2.0 > Migrate flink-connectors-aws to new structure >

[GitHub] [flink-connector-aws] dannycranmer merged pull request #62: [FLINK-31662][connectors][aws] Restructure repository to account for …

2023-06-30 Thread via GitHub
dannycranmer merged PR #62: URL: https://github.com/apache/flink-connector-aws/pull/62 -- 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:

[jira] [Updated] (FLINK-32262) Add MAP_ENTRIES support in SQL & Table API

2023-06-30 Thread Hanyu Zheng (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32262?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Hanyu Zheng updated FLINK-32262: Description: Implement the {{map_entries}} method to transform a map into an array of key-value

[jira] [Created] (FLINK-32507) Document KafkaSink SinkWriterMetricGroup metrics

2023-06-30 Thread Mason Chen (Jira)
Mason Chen created FLINK-32507: -- Summary: Document KafkaSink SinkWriterMetricGroup metrics Key: FLINK-32507 URL: https://issues.apache.org/jira/browse/FLINK-32507 Project: Flink Issue Type:

[jira] [Commented] (FLINK-32496) Sources with idleness and alignment always wait for alignment when part of multiple sources is idle

2023-06-30 Thread Rui Fan (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739192#comment-17739192 ] Rui Fan commented on FLINK-32496: - Thanks [~haishui]  and [~mason6345] ,  I will finish it asap.:) >

[GitHub] [flink] 1996fanrui commented on pull request #22921: [FLINK-32506][connectors/common] Add the benchmark for watermark aggregation when enabling the watermark alignment

2023-06-30 Thread via GitHub
1996fanrui commented on PR #22921: URL: https://github.com/apache/flink/pull/22921#issuecomment-1614986008 Hi @pnowojski @RocMarshal , I have finished the first version of watermark aggregation benchmark. Would you mind help take a look in your free time? thanks~

[jira] [Assigned] (FLINK-32422) EmbeddedLeaderService doesn't handle the leader events properly in edge cases

2023-06-30 Thread Matthias Pohl (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32422?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias Pohl reassigned FLINK-32422: - Assignee: Matthias Pohl > EmbeddedLeaderService doesn't handle the leader events

[jira] [Assigned] (FLINK-31837) Move LeaderElectionDriver instantiated into DefaultLeaderElectionService.register to instantiate the driver lazily

2023-06-30 Thread Matthias Pohl (Jira)
[ https://issues.apache.org/jira/browse/FLINK-31837?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias Pohl reassigned FLINK-31837: - Assignee: Matthias Pohl > Move LeaderElectionDriver instantiated into >

[jira] [Assigned] (FLINK-32503) FLIP-285 technical debt

2023-06-30 Thread Matthias Pohl (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32503?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias Pohl reassigned FLINK-32503: - Assignee: Matthias Pohl > FLIP-285 technical debt > --- > >

[jira] [Commented] (FLINK-32496) Sources with idleness and alignment always wait for alignment when part of multiple sources is idle

2023-06-30 Thread Mason Chen (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739189#comment-17739189 ] Mason Chen commented on FLINK-32496: Hi [~haishui] [~fanrui]! I was looking at the user thread too

[GitHub] [flink] flinkbot commented on pull request #22921: [FLINK-32506][connectors/common] Add the benchmark for watermark aggregation when enabling the watermark alignment

2023-06-30 Thread via GitHub
flinkbot commented on PR #22921: URL: https://github.com/apache/flink/pull/22921#issuecomment-1614960838 ## CI report: * 73edc3a766af1d716a28c63b6729042ac94eb12d UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[jira] [Updated] (FLINK-32506) Add the watermark aggregation benchmark for source coordinator

2023-06-30 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32506?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated FLINK-32506: --- Labels: pull-request-available (was: ) > Add the watermark aggregation benchmark for

[GitHub] [flink] 1996fanrui opened a new pull request, #22921: [FLINK-32506][connectors/common] Add the benchmark for watermark aggregation when enabling the watermark alignment

2023-06-30 Thread via GitHub
1996fanrui opened a new pull request, #22921: URL: https://github.com/apache/flink/pull/22921 ## What is the purpose of the change [FLINK-32420](https://issues.apache.org/jira/browse/FLINK-32420) is improving the watermark aggregation performance. We want to add a benchmark

[GitHub] [flink] hanyuzheng7 commented on pull request #22785: [FLINK-32259][table] Add built-in ARRAY_JOIN function.

2023-06-30 Thread via GitHub
hanyuzheng7 commented on PR #22785: URL: https://github.com/apache/flink/pull/22785#issuecomment-1614890408 @flinkbot run Azure -- 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

[jira] [Created] (FLINK-32506) Add the watermark aggregation benchmark for source coordinator

2023-06-30 Thread Rui Fan (Jira)
Rui Fan created FLINK-32506: --- Summary: Add the watermark aggregation benchmark for source coordinator Key: FLINK-32506 URL: https://issues.apache.org/jira/browse/FLINK-32506 Project: Flink Issue

[jira] [Resolved] (FLINK-32505) Compilation error in ProducerMergedPartitionFileWriter

2023-06-30 Thread Matthias Pohl (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32505?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias Pohl resolved FLINK-32505. --- Fix Version/s: 1.18.0 Resolution: Fixed master:

[GitHub] [flink] XComp merged pull request #22920: [FLINK-32505][network] Fixed compilation error in ProducerMergedPartitionFileWriter.

2023-06-30 Thread via GitHub
XComp merged PR #22920: URL: https://github.com/apache/flink/pull/22920 -- 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:

[GitHub] [flink] XComp commented on pull request #22920: [FLINK-32505][network] Fixed compilation error in ProducerMergedPartitionFileWriter.

2023-06-30 Thread via GitHub
XComp commented on PR #22920: URL: https://github.com/apache/flink/pull/22920#issuecomment-1614874864 thanks for the quick approval. A local run succeeded. I'm not gonna wait for CI and merge the fix rightaway to make `master` succeed again, hopefully. -- This is an automated message

[GitHub] [flink] XComp commented on pull request #22844: [FLINK-31784][runtime] Adds multi-component support to DefaultLeaderElectionService

2023-06-30 Thread via GitHub
XComp commented on PR #22844: URL: https://github.com/apache/flink/pull/22844#issuecomment-1614873661 and another one after I rebased FLINK-31783 again -.- -- 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

[GitHub] [flink] reswqa commented on pull request #22920: [FLINK-32505][network] Fixed compilation error in ProducerMergedPartitionFileWriter.

2023-06-30 Thread via GitHub
reswqa commented on PR #22920: URL: https://github.com/apache/flink/pull/22920#issuecomment-1614862202 Thanks for the fix! -- 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.

[GitHub] [flink] flinkbot commented on pull request #22920: [FLINK-32505][network] Fixed compilation error in ProducerMergedPartitionFileWriter.

2023-06-30 Thread via GitHub
flinkbot commented on PR #22920: URL: https://github.com/apache/flink/pull/22920#issuecomment-1614856839 ## CI report: * 6c1d7488d10adf01b8d609b38eadf8c0855a09a5 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[GitHub] [flink-connector-gcp-pubsub] jayadeep-jayaraman commented on pull request #15: [FLINK-32031] Flink GCP Connector having issues with Conscrypt library

2023-06-30 Thread via GitHub
jayadeep-jayaraman commented on PR #15: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/15#issuecomment-1614855328 @MartijnVisser - I closed the earlier PR and raised a new one, my local workspace had got corrupted so I cleaned it up and added back the changes. Please take

[GitHub] [flink] XComp commented on pull request #22661: [FLINK-31783][runtime] Migrates DefaultLeaderElectionService from LeaderElectionDriver to the MultipleComponentLeaderElectionDriver interface

2023-06-30 Thread via GitHub
XComp commented on PR #22661: URL: https://github.com/apache/flink/pull/22661#issuecomment-1614853966 I had to do another round because of FLINK-32505 :roll_eyes: I now force-pushed the branch based on 1ef997369799b1b72d263c945ddeafcf671b3326 which had a green CI run in master -- This

[GitHub] [flink] XComp commented on pull request #22920: [FLINK-32505][network] Fixed compilation error in ProducerMergedPartitionFileWriter.

2023-06-30 Thread via GitHub
XComp commented on PR #22920: URL: https://github.com/apache/flink/pull/22920#issuecomment-1614844835 @reswqa can you approve this issue to fix master? -- 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

[GitHub] [flink] chucheng92 commented on a diff in pull request #21442: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #21442: URL: https://github.com/apache/flink/pull/21442#discussion_r1248018536 ## pom.xml: ## @@ -684,9 +684,9 @@ under the License. - commons-collections

[jira] [Updated] (FLINK-32505) Compilation error in ProducerMergedPartitionFileWriter

2023-06-30 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32505?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated FLINK-32505: --- Labels: pull-request-available (was: ) > Compilation error in

[GitHub] [flink] XComp opened a new pull request, #22920: [FLINK-32505][network] Fixed compilation error in ProducerMergedPartitionFileWriter.

2023-06-30 Thread via GitHub
XComp opened a new pull request, #22920: URL: https://github.com/apache/flink/pull/22920 Looks like the issue was caused by the introduction of FLINK-31644 and the concurrent upgrade of flink-shaded. ## What is the purpose of the change Fixes the compilation error ##

[jira] [Commented] (FLINK-32208) Remove dependency on flink-shaded from flink-connector-aws

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17739142#comment-17739142 ] Danny Cranmer commented on FLINK-32208: --- merged commit 

[jira] [Resolved] (FLINK-32208) Remove dependency on flink-shaded from flink-connector-aws

2023-06-30 Thread Danny Cranmer (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32208?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Danny Cranmer resolved FLINK-32208. --- Resolution: Fixed > Remove dependency on flink-shaded from flink-connector-aws >

[GitHub] [flink-connector-aws] dannycranmer merged pull request #85: [FLINK-32208] Remove dependency on flink-shaded from flink-connector-aws

2023-06-30 Thread via GitHub
dannycranmer merged PR #85: URL: https://github.com/apache/flink-connector-aws/pull/85 -- 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:

[jira] [Created] (FLINK-32505) Compilation error in ProducerMergedPartitionFileWriter

2023-06-30 Thread Matthias Pohl (Jira)
Matthias Pohl created FLINK-32505: - Summary: Compilation error in ProducerMergedPartitionFileWriter Key: FLINK-32505 URL: https://issues.apache.org/jira/browse/FLINK-32505 Project: Flink

[GitHub] [flink-connector-gcp-pubsub] jayadeep-jayaraman closed pull request #13: [FLINK-32031] Flink GCP Connector having issues with Conscrypt library

2023-06-30 Thread via GitHub
jayadeep-jayaraman closed pull request #13: [FLINK-32031] Flink GCP Connector having issues with Conscrypt library URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/13 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [flink-connector-aws] dannycranmer commented on a diff in pull request #85: [FLINK-32208] Remove dependency on flink-shaded from flink-connector-aws

2023-06-30 Thread via GitHub
dannycranmer commented on code in PR #85: URL: https://github.com/apache/flink-connector-aws/pull/85#discussion_r1248007942 ## flink-connector-dynamodb/src/main/java/org/apache/flink/connector/dynamodb/table/DynamoDbDynamicSinkFactory.java: ## @@ -72,6 +70,10 @@ public String

[GitHub] [flink-connector-kafka] chucheng92 commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r1248008592 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment: >

[GitHub] [flink-connector-kafka] chucheng92 commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r1248008592 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment: >

[GitHub] [flink] dannycranmer commented on a diff in pull request #21458: [FLINK-28513] Fix Flink Table API CSV streaming sink throws SerializedThrowable exception

2023-06-30 Thread via GitHub
dannycranmer commented on code in PR #21458: URL: https://github.com/apache/flink/pull/21458#discussion_r1248002523 ## flink-filesystems/flink-s3-fs-base/src/main/java/org/apache/flink/fs/s3/common/writer/S3RecoverableFsDataOutputStream.java: ## @@ -126,7 +126,16 @@ public long

[GitHub] [flink-connector-kafka] chucheng92 commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r1247988699 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment:

[GitHub] [flink-connector-gcp-pubsub] MartijnVisser commented on pull request #13: [FLINK-32031] Flink GCP Connector having issues with Conscrypt library

2023-06-30 Thread via GitHub
MartijnVisser commented on PR #13: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/13#issuecomment-1614819302 @jayadeep-jayaraman I'm a bit surprised that d7ed02b76be15afdff0f9c8fc0805bc30ce6c559 is now appearing in this PR as a commit, that shouldn't have been the case

[GitHub] [flink] flinkbot commented on pull request #22919: [FLINK-31837][runtime] Makes LeaderElectionDriver instantiation happen lazily

2023-06-30 Thread via GitHub
flinkbot commented on PR #22919: URL: https://github.com/apache/flink/pull/22919#issuecomment-1614819532 ## CI report: * 2a64d561b88bdf51aec952e557d08310e05189fa UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[GitHub] [flink] XComp commented on pull request #22844: [FLINK-31784][runtime] Adds multi-component support to DefaultLeaderElectionService

2023-06-30 Thread via GitHub
XComp commented on PR #22844: URL: https://github.com/apache/flink/pull/22844#issuecomment-1614818534 Force-pushed a rebase to the most-recent version of [FLINK-31783](https://issues.apache.org/jira/browse/FLINK-31783) to include `master` which appears to be more stable now -- This is

[GitHub] [flink-connector-kafka] chucheng92 commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r1247988699 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment:

[GitHub] [flink] dannycranmer commented on a diff in pull request #22906: [FLINK-32488][REST] Introduce separate configuration to control expiry of E…

2023-06-30 Thread via GitHub
dannycranmer commented on code in PR #22906: URL: https://github.com/apache/flink/pull/22906#discussion_r1247985816 ## docs/layouts/shortcodes/generated/expert_rest_section.html: ## @@ -20,6 +20,12 @@ Long The time in ms that the client waits for the

[GitHub] [flink] XComp commented on pull request #22661: [FLINK-31783][runtime] Migrates DefaultLeaderElectionService from LeaderElectionDriver to the MultipleComponentLeaderElectionDriver interface

2023-06-30 Thread via GitHub
XComp commented on PR #22661: URL: https://github.com/apache/flink/pull/22661#issuecomment-1614816466 Force-pushed rebase since master seems to be more stable. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [flink-connector-kafka] chucheng92 commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r124799 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment:

[GitHub] [flink-connector-kafka] chucheng92 commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r124799 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment:

[GitHub] [flink-connector-kafka] chucheng92 commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r124799 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment:

[GitHub] [flink-connector-kafka] chucheng92 commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r124799 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment:

[jira] [Updated] (FLINK-31837) Move LeaderElectionDriver instantiated into DefaultLeaderElectionService.register to instantiate the driver lazily

2023-06-30 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/FLINK-31837?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated FLINK-31837: --- Labels: pull-request-available (was: ) > Move LeaderElectionDriver instantiated into >

[GitHub] [flink] XComp opened a new pull request, #22919: [FLINK-31837][runtime] Makes LeaderElectionDriver instantiation happen lazily

2023-06-30 Thread via GitHub
XComp opened a new pull request, #22919: URL: https://github.com/apache/flink/pull/22919 ## What is the purpose of the change The driver doesn't need to be instantiated when the `DefaultLeaderElectionService` is instantiated. Instead, we can instantiated when the first contender is

[GitHub] [flink-connector-kafka] chucheng92 commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
chucheng92 commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r1247988699 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment:

[jira] [Updated] (FLINK-31816) Refactor EmbeddedLeaderElectionService

2023-06-30 Thread Matthias Pohl (Jira)
[ https://issues.apache.org/jira/browse/FLINK-31816?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias Pohl updated FLINK-31816: -- Parent: FLINK-32503 Issue Type: Sub-task (was: Technical Debt) > Refactor

[GitHub] [flink-connector-kafka] MartijnVisser commented on a diff in pull request #37: [FLINK-30274] Bump commons-collections 3.x to commons-collections4

2023-06-30 Thread via GitHub
MartijnVisser commented on code in PR #37: URL: https://github.com/apache/flink-connector-kafka/pull/37#discussion_r1247980121 ## flink-connector-kafka/pom.xml: ## @@ -72,6 +72,13 @@ under the License. ${kafka.version} + Review Comment:

[jira] [Updated] (FLINK-30789) Merge MultipleComponentsLeaderElectionDriver methods notifyAllKnownLeaderInformation and notifyLeaderInformationChange into a single method and

2023-06-30 Thread Matthias Pohl (Jira)
[ https://issues.apache.org/jira/browse/FLINK-30789?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias Pohl updated FLINK-30789: -- Parent: FLINK-32503 Issue Type: Sub-task (was: Technical Debt) > Merge

[GitHub] [flink] hanyuzheng7 commented on a diff in pull request #22917: [FLINK-32498][table] Fix the problem of array_max return type, it sho…

2023-06-30 Thread via GitHub
hanyuzheng7 commented on code in PR #22917: URL: https://github.com/apache/flink/pull/22917#discussion_r1247961308 ## flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/inference/strategies/SpecificTypeStrategies.java: ## @@ -58,8 +58,12 @@ public final

[jira] [Updated] (FLINK-32157) Replaces LeaderConnectionInfo with LeaderInformation

2023-06-30 Thread Matthias Pohl (Jira)
[ https://issues.apache.org/jira/browse/FLINK-32157?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias Pohl updated FLINK-32157: -- Parent: (was: FLINK-26522) Issue Type: Technical Debt (was: Sub-task) > Replaces

  1   2   3   4   >