Re: [PR] Adds integration testing for blocklist handling [cassandra-analytics]

2024-01-17 Thread via GitHub
arjunashok commented on code in PR #32: URL: https://github.com/apache/cassandra-analytics/pull/32#discussion_r1456276685 ## cassandra-analytics-integration-tests/src/test/java/org/apache/cassandra/analytics/BlockedInstancesTest.java: ## @@ -0,0 +1,167 @@ +/* + * Licensed to

Re: [PR] Adds integration testing for blocklist handling [cassandra-analytics]

2024-01-17 Thread via GitHub
frankgh commented on code in PR #32: URL: https://github.com/apache/cassandra-analytics/pull/32#discussion_r1455673981 ## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/WriterOptions.java: ## @@ -50,4 +50,5 @@ public enum WriterOptions implements

Re: [PR] Adds integration testing for blocklist handling [cassandra-analytics]

2024-01-16 Thread via GitHub
frankgh commented on code in PR #32: URL: https://github.com/apache/cassandra-analytics/pull/32#discussion_r1454284104 ## cassandra-analytics-integration-tests/src/test/java/org/apache/cassandra/analytics/BlockedInstancesTest.java: ## @@ -0,0 +1,167 @@ +/* + * Licensed to the

[PR] Adds integration testing for blocklist handling [cassandra-analytics]

2024-01-16 Thread via GitHub
arjunashok opened a new pull request, #32: URL: https://github.com/apache/cassandra-analytics/pull/32 ### Changes - Adds spark config with default empty list for `blocked_instances` - Adds tests for blocked instances -- This is an automated message from the Apache Git Service. To