[ 
https://issues.apache.org/jira/browse/CASSANDRA-20328?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17952596#comment-17952596
 ] 

Stefan Miklosovic edited comment on CASSANDRA-20328 at 5/19/25 1:18 PM:
------------------------------------------------------------------------

To avoid these issues in the future I would add to checkstyle that 
"BulkLoader.class: is forbidden to use in server's code (or that 
org.apache.cassandra.tools.BulkLoader import in server code is forbidden as 
well or something like that). 

I am not sure how this works in Ant exactly but we might contemplate to not add 
whole "sstableloader" to core's classpath. Basically, it would not be even 
possible to import BulkLoader in server's code anywhere.


was (Author: smiklosovic):
To avoid these issues in the future I would add to checkstyle that 
"BulkLoader.class: is forbidden to use in server's code (or that 
org.apache.cassandra.tools.BulkLoader import in server code is forbidden as 
well or something like that). 

> Migrate sstableloader code to its own tools directory
> -----------------------------------------------------
>
>                 Key: CASSANDRA-20328
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-20328
>             Project: Apache Cassandra
>          Issue Type: Improvement
>          Components: Tool/sstable
>            Reporter: Andy Tolbert
>            Assignee: Andy Tolbert
>            Priority: Normal
>              Labels: pull-request-available
>             Fix For: 5.x
>
>         Attachments: CASSANDRA-0413-20328_ci_summary.html, 
> CASSANDRA-20328-511-0_ci_summary.html, CASSANDRA-20328-515-0_ci_summary.html, 
> CASSANDRA-20328-test_results.html, 
> ci_summary_tolbertam_CASSANDRA-20328_238.html, 
> results_details_tolbertam_CASSANDRA-20328_238.tar.xz
>
>          Time Spent: 15h 10m
>  Remaining Estimate: 0h
>
> Akin to fqltool and stress, we should move sstableloader to its own tools 
> directory.  This is needed to decouple the java driver as a dependency from 
> server code ([CASSANDRA-20326]).
> For backwards compatibility reasons, we should retain {{bin/sstableloader}}.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to