Re: [PR] [FLINK-20398][e2e] Migrate test_batch_sql.sh to Java e2e tests framework [flink]

2024-06-09 Thread via GitHub
XComp commented on code in PR #24471: URL: https://github.com/apache/flink/pull/24471#discussion_r1632637479 ## flink-end-to-end-tests/flink-batch-sql-test/src/test/java/org/apache/flink/sql/tests/Generator.java: ## @@ -0,0 +1,82 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] [FLINK-35528][task] Skip execution of interruptible mails when yielding [flink]

2024-06-09 Thread via GitHub
Zakelly commented on code in PR #24904: URL: https://github.com/apache/flink/pull/24904#discussion_r1632632895 ## flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/mailbox/Mail.java: ## @@ -71,8 +77,13 @@ public Mail( this.actionExecutor = acti

Re: [PR] [FLINK-20217][task] Allow certains operators to yield to unaligned checkpoint in case timers are firing [flink]

2024-06-09 Thread via GitHub
Zakelly commented on code in PR #24895: URL: https://github.com/apache/flink/pull/24895#discussion_r1632623556 ## flink-streaming-java/src/main/java/org/apache/flink/streaming/api/operators/InternalTimerServiceImpl.java: ## @@ -307,18 +307,36 @@ void onProcessingTime(long time)

Re: [PR] [FLINK-20217][task] Allow certains operators to yield to unaligned checkpoint in case timers are firing [flink]

2024-06-09 Thread via GitHub
Zakelly commented on code in PR #24895: URL: https://github.com/apache/flink/pull/24895#discussion_r1632622930 ## flink-streaming-java/src/main/java/org/apache/flink/streaming/api/operators/InternalTimeServiceManager.java: ## @@ -40,6 +40,15 @@ */ @Internal public interface

Re: [PR] [FLINK-35371][security] Add configuration for SSL keystore and truststore type [flink]

2024-06-09 Thread via GitHub
flinkbot commented on PR #24919: URL: https://github.com/apache/flink/pull/24919#issuecomment-2157312543 ## CI report: * 14cf0fc58dfb9553ffe703622b6223a54549aacd UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[PR] [FLINK-35371][security] Add configuration for SSL keystore and truststore type [flink]

2024-06-09 Thread via GitHub
ammar-master opened a new pull request, #24919: URL: https://github.com/apache/flink/pull/24919 ## What is the purpose of the change Additional configuration options for internal and REST SSL setup. ## Brief change log * SSL configuration option for intern

Re: [PR] [FLINK-35371][security] Add configuration for SSL keystore and truststore type [flink]

2024-06-09 Thread via GitHub
master-ammar closed pull request #24918: [FLINK-35371][security] Add configuration for SSL keystore and truststore type URL: https://github.com/apache/flink/pull/24918 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [PR] [FLINK-35371][security] Add configuration for SSL keystore and truststore type [flink]

2024-06-09 Thread via GitHub
flinkbot commented on PR #24918: URL: https://github.com/apache/flink/pull/24918#issuecomment-2157270012 ## CI report: * db7977869fe099d7cf1cf428bec048e608dbdd60 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[jira] [Updated] (FLINK-35371) Allow the keystore and truststore type to configured for SSL

2024-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/FLINK-35371?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated FLINK-35371: --- Labels: SSL pull-request-available (was: SSL) > Allow the keystore and truststore type to c

[PR] [FLINK-35371][security] Add configuration for SSL keystore and truststore type [flink]

2024-06-09 Thread via GitHub
master-ammar opened a new pull request, #24918: URL: https://github.com/apache/flink/pull/24918 ## What is the purpose of the change Additional configuration options for internal and REST SSL setup. ## Brief change log * SSL configuration option for in

Re: [PR] [FLINK-34108][table] Add built-in URL_ENCODE and URL_DECODE function. [flink]

2024-06-09 Thread via GitHub
superdiaodiao commented on PR #24773: URL: https://github.com/apache/flink/pull/24773#issuecomment-2157065404 > Thanks for the contribution @superdiaodiao > thanks for the review @HuangXingBo , @davidradl > > it looks ok to me, I will test it a bit more and in case of succeed will

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
vahmed-hamdy commented on PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#issuecomment-2156823709 @jeyhunkarimov thanks for the feedback, I added a negative test as well @snuyanzin Thanks for the review, All comments addressed as well -- This is an automated m

Re: [PR] [FLINK-34108][table] Add built-in URL_ENCODE and URL_DECODE function. [flink]

2024-06-09 Thread via GitHub
snuyanzin commented on PR #24773: URL: https://github.com/apache/flink/pull/24773#issuecomment-2156817722 Thanks for the contribution @superdiaodiao thanks for the review @HuangXingBo , @davidradl it looks ok to me, I will test it a bit more and in case of succeed will merge it

[jira] [Commented] (FLINK-35520) master can't compile as license check failed

2024-06-09 Thread Sergey Nuyanzin (Jira)
[ https://issues.apache.org/jira/browse/FLINK-35520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17853540#comment-17853540 ] Sergey Nuyanzin commented on FLINK-35520: - That seem to be trickier than I thoug

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
vahmed-hamdy commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632401145 ## flink-connector-gcp-pubsub/src/main/java/org/apache/flink/streaming/connectors/gcp/pubsub/emulator/EmulatorCredentialsProvider.java: ## @@ -20,1

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
vahmed-hamdy commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632400893 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/PubSubSinkV2ITTests.java: ## @@ -0,0 +1,114 @@ +pa

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
vahmed-hamdy commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632400806 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/PubSubSinkV2ITTests.java: ## @@ -0,0 +1,114 @@ +pa

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
snuyanzin commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632391517 ## flink-connector-gcp-pubsub/src/main/java/org/apache/flink/streaming/connectors/gcp/pubsub/emulator/EmulatorCredentialsProvider.java: ## @@ -20,12 +

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
snuyanzin commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632391131 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/PubSubSinkV2ITTests.java: ## @@ -0,0 +1,114 @@ +packa

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
snuyanzin commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632390729 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/PubSubSinkV2ITTests.java: ## @@ -0,0 +1,114 @@ +packa

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
snuyanzin commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632390518 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/PubSubSinkV2ITTests.java: ## @@ -0,0 +1,115 @@ +packa

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
snuyanzin commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632390394 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/PubSubSinkV2ITTests.java: ## @@ -0,0 +1,114 @@ +packa

Re: [PR] [FLINK-35305]Amazon SQS Sink Connector [flink-connector-aws]

2024-06-09 Thread via GitHub
19priyadhingra commented on code in PR #141: URL: https://github.com/apache/flink-connector-aws/pull/141#discussion_r1632385233 ## flink-connector-aws/flink-connector-sqs/src/main/java/org.apache.flink.connector.sqs/sink/SqsSinkWriter.java: ## @@ -0,0 +1,257 @@ +/* + * Licensed

Re: [PR] [FLINK-35305]Amazon SQS Sink Connector [flink-connector-aws]

2024-06-09 Thread via GitHub
19priyadhingra commented on code in PR #141: URL: https://github.com/apache/flink-connector-aws/pull/141#discussion_r1632370742 ## flink-connector-aws/flink-connector-sqs/src/main/java/org.apache.flink.connector.sqs/sink/SqsConfigConstants.java: ## @@ -0,0 +1,31 @@ +/* + * Lice

Re: [PR] [FLINK-34977][API] Introduce State Access on DataStream API V2 [flink]

2024-06-09 Thread via GitHub
jeyhunkarimov commented on PR #24725: URL: https://github.com/apache/flink/pull/24725#issuecomment-2156736006 Hi @reswqa thanks a lot for the review. Could you please do another pass in your available time? Thanks! -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] [Flink-35473][table] Improve Table/SQL Configuration for Flink 2.0 [flink]

2024-06-09 Thread via GitHub
LadyForest commented on code in PR #24889: URL: https://github.com/apache/flink/pull/24889#discussion_r1632297334 ## flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/api/config/LookupJoinHintOptions.java: ## @@ -0,0 +1,136 @@ +/* + * Licensed to the Apache S

[jira] [Comment Edited] (FLINK-34108) Add URL_ENCODE and URL_DECODE function

2024-06-09 Thread chesterxu (Jira)
[ https://issues.apache.org/jira/browse/FLINK-34108?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17853489#comment-17853489 ] chesterxu edited comment on FLINK-34108 at 6/9/24 12:34 PM:

[jira] [Commented] (FLINK-34108) Add URL_ENCODE and URL_DECODE function

2024-06-09 Thread chesterxu (Jira)
[ https://issues.apache.org/jira/browse/FLINK-34108?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17853489#comment-17853489 ] chesterxu commented on FLINK-34108: --- [~martijnvisser]  Please take a CR, thanks! > Ad

[jira] [Updated] (FLINK-35560) Add query validator support to flink sql gateway via spi pattern

2024-06-09 Thread dongwoo.kim (Jira)
[ https://issues.apache.org/jira/browse/FLINK-35560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] dongwoo.kim updated FLINK-35560: Description: h3. Summary Hello I'd like to suggest query validator support in flink sql gateway v

[jira] [Updated] (FLINK-35560) Add query validator support to flink sql gateway via spi pattern

2024-06-09 Thread dongwoo.kim (Jira)
[ https://issues.apache.org/jira/browse/FLINK-35560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] dongwoo.kim updated FLINK-35560: Description: h3. Summary Hello I'd like to suggest query validator support in flink sql gateway v

[jira] [Updated] (FLINK-35560) Add query validator support to flink sql gateway via spi pattern

2024-06-09 Thread dongwoo.kim (Jira)
[ https://issues.apache.org/jira/browse/FLINK-35560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] dongwoo.kim updated FLINK-35560: Description: h3. Summary Hello I'd like to suggest query validator support in flink sql gateway v

[jira] [Created] (FLINK-35560) Add query validator support to flink sql gateway via spi pattern

2024-06-09 Thread dongwoo.kim (Jira)
dongwoo.kim created FLINK-35560: --- Summary: Add query validator support to flink sql gateway via spi pattern Key: FLINK-35560 URL: https://issues.apache.org/jira/browse/FLINK-35560 Project: Flink

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
vahmed-hamdy commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632243987 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/util/PubsubHelper.java: ## @@ -36,27 +40,50 @@ im

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
vahmed-hamdy commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632243692 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/util/PubsubHelper.java: ## @@ -36,27 +40,50 @@ im

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
vahmed-hamdy commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632243045 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/PubSubSinkV2ITTests.java: ## @@ -0,0 +1,115 @@ +pa

Re: [PR] [FLINK-35548] Add E2E tests for PubSubSinkV2 [flink-connector-gcp-pubsub]

2024-06-09 Thread via GitHub
vahmed-hamdy commented on code in PR #28: URL: https://github.com/apache/flink-connector-gcp-pubsub/pull/28#discussion_r1632242928 ## flink-connector-gcp-pubsub-e2e-tests/src/test/java/org/apache/flink/connector/gcp/pubsub/sink/PubSubSinkV2ITTests.java: ## @@ -0,0 +1,115 @@ +pa

Re: [PR] [FLINK-33138] DataStream API implementation [flink-connector-prometheus]

2024-06-09 Thread via GitHub
nicusX commented on PR #1: URL: https://github.com/apache/flink-connector-prometheus/pull/1#issuecomment-2156438875 Addressed the latest comments and ported AMP signer to AWS SDKv2 (!) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[jira] [Commented] (FLINK-35138) Release flink-connector-kafka v3.2.0 for Flink 1.19

2024-06-09 Thread Tamir Sagi (Jira)
[ https://issues.apache.org/jira/browse/FLINK-35138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17853470#comment-17853470 ] Tamir Sagi commented on FLINK-35138: Thank you Danny! highly appreciated it. > Rele

[jira] [Comment Edited] (FLINK-35138) Release flink-connector-kafka v3.2.0 for Flink 1.19

2024-06-09 Thread Tamir Sagi (Jira)
[ https://issues.apache.org/jira/browse/FLINK-35138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17853470#comment-17853470 ] Tamir Sagi edited comment on FLINK-35138 at 6/9/24 7:45 AM:

[jira] [Commented] (FLINK-35138) Release flink-connector-kafka v3.2.0 for Flink 1.19

2024-06-09 Thread Tamir Sagi (Jira)
[ https://issues.apache.org/jira/browse/FLINK-35138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17853471#comment-17853471 ] Tamir Sagi commented on FLINK-35138: Thank you Danny! highly appreciated. > Release

[jira] (FLINK-35138) Release flink-connector-kafka v3.2.0 for Flink 1.19

2024-06-09 Thread Tamir Sagi (Jira)
[ https://issues.apache.org/jira/browse/FLINK-35138 ] Tamir Sagi deleted comment on FLINK-35138: was (Author: JIRAUSER283777): Thank you Danny! highly appreciated. > Release flink-connector-kafka v3.2.0 for Flink 1.19 >

Re: [PR] [FLINK-33138] DataStream API implementation [flink-connector-prometheus]

2024-06-09 Thread via GitHub
nicusX commented on code in PR #1: URL: https://github.com/apache/flink-connector-prometheus/pull/1#discussion_r1632187848 ## README.md: ## @@ -8,6 +8,14 @@ Apache Flink is an open source stream processing framework with powerful stream- Learn more about Flink at [https://

Re: [PR] [FLINK-33138] DataStream API implementation [flink-connector-prometheus]

2024-06-09 Thread via GitHub
nicusX commented on code in PR #1: URL: https://github.com/apache/flink-connector-prometheus/pull/1#discussion_r1632194796 ## amp-request-signer/README.md: ## @@ -0,0 +1,31 @@ +## Request Signer for Amazon Managed Prometheus (AMP) + +Request signer implementation for Amazon Man

Re: [PR] [FLINK-33138] DataStream API implementation [flink-connector-prometheus]

2024-06-09 Thread via GitHub
nicusX commented on code in PR #1: URL: https://github.com/apache/flink-connector-prometheus/pull/1#discussion_r1632194645 ## README.md: ## @@ -8,6 +8,14 @@ Apache Flink is an open source stream processing framework with powerful stream- Learn more about Flink at [https://

Re: [PR] [FLINK-33138] DataStream API implementation [flink-connector-prometheus]

2024-06-09 Thread via GitHub
nicusX commented on code in PR #1: URL: https://github.com/apache/flink-connector-prometheus/pull/1#discussion_r1632188645 ## README.md: ## @@ -8,6 +8,14 @@ Apache Flink is an open source stream processing framework with powerful stream- Learn more about Flink at [https://