lukasz-antoniak commented on code in PR #169:
URL:
https://github.com/apache/cassandra-analytics/pull/169#discussion_r3544349005
##########
cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/bulkwriter/BulkSparkConf.java:
##########
@@ -114,9 +114,9 @@ public class BulkSparkConf implements Serializable
// which will throw a configuration exception for each setting with
that prefix it does not recognize
public static final String SETTING_PREFIX = "spark.cassandra_analytics.";
- // Cassandra version of target cluster. Configuration parameter is exposed
to be able to correctly initialize static
- // components, before cluster version is discovered via Sidecar.
- public static final String CASSANDRA_VERSION =
SETTING_PREFIX + "cassandra.version";
+ // Disable SSTable version-based bridge determination. When true, falls
back to legacy mode.
Review Comment:
NIT: Maybe just to be clear:
`Disable SSTable version-based bridge determination. When true, falls back
to legacy mode, which selects bridge based on Cassandra version returned by the
Sidecar.`
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
