Re: [PR] fix flaky JdbcProjectionTest [incubator-pekko-projection]

2024-02-19 Thread via GitHub
Roiocam commented on PR #108: URL: https://github.com/apache/incubator-pekko-projection/pull/108#issuecomment-1953596766 #103 break the CI check -- 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 t

Re: [PR] fix flaky JdbcProjectionTest [incubator-pekko-projection]

2024-02-19 Thread via GitHub
Roiocam commented on code in PR #108: URL: https://github.com/apache/incubator-pekko-projection/pull/108#discussion_r1495301749 ## jdbc/src/test/java/org/apache/pekko/projection/jdbc/JdbcProjectionTest.java: ## @@ -346,8 +346,9 @@ public void atLeastOnceShouldRestartFromPreviou

[PR] fix flaky JdbcProjectionTest [incubator-pekko-projection]

2024-02-19 Thread via GitHub
Roiocam opened a new pull request, #108: URL: https://github.com/apache/incubator-pekko-projection/pull/108 # Motivation solved: #104 It is the same problem as #105. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] fix Kafka flaky test [incubator-pekko-projection]

2024-02-19 Thread via GitHub
Roiocam commented on code in PR #105: URL: https://github.com/apache/incubator-pekko-projection/pull/105#discussion_r1495297199 ## kafka-test/src/test/scala/org/apache/pekko/projection/kafka/internal/KafkaSourceProviderImplSpec.scala: ## @@ -109,6 +110,12 @@ class KafkaSourcePr

Re: [PR] fix Kafka flaky test [incubator-pekko-projection]

2024-02-19 Thread via GitHub
Roiocam commented on code in PR #105: URL: https://github.com/apache/incubator-pekko-projection/pull/105#discussion_r1495290031 ## kafka-test/src/test/scala/org/apache/pekko/projection/kafka/internal/KafkaSourceProviderImplSpec.scala: ## @@ -109,6 +110,12 @@ class KafkaSourcePr

Re: [PR] slightly better support for Solr 9 client - (but full support won't be possible until we use Java 11 to build) [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
mdedetrich commented on code in PR #481: URL: https://github.com/apache/incubator-pekko-connectors/pull/481#discussion_r1495263312 ## solr/src/main/scala/org/apache/pekko/stream/connectors/solr/impl/SolrFlowStage.scala: ## @@ -114,8 +113,12 @@ private final class SolrFlowLogic[

Re: [PR] [couchbase] feat: migrate couchbase client2 to client3 [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
laglangyue commented on PR #504: URL: https://github.com/apache/incubator-pekko-connectors/pull/504#issuecomment-1953438323 I has not finished test now, sumit pr to talk about the api changes desgin. If api design is ok,I will finished the test as soon as possible -- This is an automated

[I] Feat: migrate couchbase client2 to client3 [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
laglangyue opened a new issue, #503: URL: https://github.com/apache/incubator-pekko-connectors/issues/503 ## Motivation and backword https://github.com/apache/incubator-pekko-connectors/pull/400 The Couchbase client2 has been deprecated for quite some time, and it's believed that m

Re: [PR] slightly better support for Solr 9 client - (but full support won't be possible until we use Java 11 to build) [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
Roiocam commented on code in PR #481: URL: https://github.com/apache/incubator-pekko-connectors/pull/481#discussion_r1495141542 ## solr/src/main/scala/org/apache/pekko/stream/connectors/solr/impl/SolrFlowStage.scala: ## @@ -114,8 +113,12 @@ private final class SolrFlowLogic[T,

Re: [PR] slightly better support for Solr 9 client - (but full support won't be possible until we use Java 11 to build) [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
pjfanning commented on code in PR #481: URL: https://github.com/apache/incubator-pekko-connectors/pull/481#discussion_r1495091144 ## solr/src/main/scala/org/apache/pekko/stream/connectors/solr/impl/SolrFlowStage.scala: ## @@ -114,8 +113,12 @@ private final class SolrFlowLogic[T

[PR] [DRAFT] test with latest spring dependencies [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
pjfanning opened a new pull request, #502: URL: https://github.com/apache/incubator-pekko-connectors/pull/502 these need Java 17 -- 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 comme

Re: [PR] slightly better support for Solr 9 client - (but full support won't be possible until we use Java 11 to build) [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
samueleresca commented on code in PR #481: URL: https://github.com/apache/incubator-pekko-connectors/pull/481#discussion_r1495084621 ## solr/src/main/scala/org/apache/pekko/stream/connectors/solr/impl/SolrFlowStage.scala: ## @@ -114,8 +113,12 @@ private final class SolrFlowLogi

Re: [PR] slightly better support for Solr 9 client - (but full support won't be possible until we use Java 11 to build) [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
pjfanning commented on PR #481: URL: https://github.com/apache/incubator-pekko-connectors/pull/481#issuecomment-1953132048 Is this something that can be merged? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] use latest pravega image [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
pjfanning merged PR #453: URL: https://github.com/apache/incubator-pekko-connectors/pull/453 -- 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: notificatio

Re: [PR] use latest pravega image [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
pjfanning commented on PR #453: URL: https://github.com/apache/incubator-pekko-connectors/pull/453#issuecomment-1952039295 I'll merge since the failure is in mqtt-streaming and that is not affected by a pravega image change. -- This is an automated message from the Apache Git Service. To

Re: [PR] Update sqs to 2.24.0 [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
pjfanning closed pull request #478: Update sqs to 2.24.0 URL: https://github.com/apache/incubator-pekko-connectors/pull/478 -- 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 u

Re: [PR] Revert "Update hbase-common, hbase-shaded-client to 1.7.2" [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
raboof commented on PR #499: URL: https://github.com/apache/incubator-pekko-connectors/pull/499#issuecomment-1951979431 > I am not an expert on Scala steward but it seems like the config changes might be wrong. I think it would work better to put these changes in the updates.ignore section

Re: [PR] use latest pravega image [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
raboof commented on PR #453: URL: https://github.com/apache/incubator-pekko-connectors/pull/453#issuecomment-1951970499 Filed #501 for the failure, re-running -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[I] MqttSessionSpec: MQTT server connector should produce a duplicate publish on the server given two client connections [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
raboof opened a new issue, #501: URL: https://github.com/apache/incubator-pekko-connectors/issues/501 Leaked children: ``` [info] MqttSessionSpec: ... [info] MQTT server connector ... [info] - should produce a duplicate publish on the server given two client connections

Re: [PR] Update kudu-client to 1.17.0 [incubator-pekko-connectors]

2024-02-19 Thread via GitHub
nvollmar commented on PR #376: URL: https://github.com/apache/incubator-pekko-connectors/pull/376#issuecomment-1951897688 on sonatype: https://central.sonatype.com/artifact/org.apache.kudu/kudu-client-tools/1.7.0 -- This is an automated message from the Apache Git Service. To respond to