jberragan commented on code in PR #88: URL: https://github.com/apache/cassandra-analytics/pull/88#discussion_r1803924924
########## cassandra-analytics-core/src/main/java/org/apache/cassandra/spark/data/DataLayer.java: ########## @@ -46,7 +46,7 @@ import org.apache.cassandra.spark.sparksql.filters.PartitionKeyFilter; import org.apache.cassandra.spark.sparksql.filters.PruneColumnFilter; import org.apache.cassandra.spark.sparksql.filters.SparkRangeFilter; -import org.apache.cassandra.spark.stats.Stats; +import org.apache.cassandra.analytics.stats.Stats; Review Comment: I don't think that is really in scope for this change. I don't want PRs to become a bucket for every change everyone wants to squeeze in. I'm tempted to revert https://github.com/apache/cassandra-analytics/pull/88/commits/67590a268f095ef6f91bd7e1d195300cd75dac71 as well. -- 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]
