[GitHub] [cassandra-analytics] yifan-c commented on a diff in pull request #9: [CASSANDRA-18633] Added Caching of Node Settings to Improve Efficiency

2023-06-29 Thread via GitHub
yifan-c commented on code in PR #9: URL: https://github.com/apache/cassandra-analytics/pull/9#discussion_r1247271887 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraClusterInfo.java: ## @@ -338,11 +346,14 @@ public String getVersionFromFe

[GitHub] [cassandra-analytics] yifan-c commented on a diff in pull request #9: [CASSANDRA-18633] Added Caching of Node Settings to Improve Efficiency

2023-06-29 Thread via GitHub
yifan-c commented on code in PR #9: URL: https://github.com/apache/cassandra-analytics/pull/9#discussion_r1247271887 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraClusterInfo.java: ## @@ -338,11 +346,14 @@ public String getVersionFromFe

[GitHub] [cassandra-analytics] yifan-c commented on a diff in pull request #9: [CASSANDRA-18633] Added Caching of Node Settings to Improve Efficiency

2023-06-29 Thread via GitHub
yifan-c commented on code in PR #9: URL: https://github.com/apache/cassandra-analytics/pull/9#discussion_r1247271478 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraClusterInfo.java: ## @@ -64,11 +65,18 @@ public class CassandraClusterInf

[GitHub] [cassandra-analytics] yifan-c commented on a diff in pull request #9: [CASSANDRA-18633] Added Caching of Node Settings to Improve Efficiency

2023-06-27 Thread via GitHub
yifan-c commented on code in PR #9: URL: https://github.com/apache/cassandra-analytics/pull/9#discussion_r1244426547 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/CassandraClusterInfo.java: ## @@ -338,11 +339,23 @@ public String getVersionFromFe