[ https://issues.apache.org/jira/browse/CASSANDRA-17677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17579785#comment-17579785 ]
Francisco Guerrero commented on CASSANDRA-17677: ------------------------------------------------ I went ahead and implemented the suggestion of handling the parameters internally in bytes. This approach allows us to have no conversion issues. Pipelines look happy for {{4.1}} but there are 15 errors here https://app.circleci.com/pipelines/github/frankgh/cassandra/110/workflows/07658b64-caea-41dc-93d2-214de0ff7f2b/jobs/2453. They seem to be coming from the {{test_collection_function}} tests. These failing tests look unrelated to this PR. > Fix BulkLoader to load entireSSTableThrottle and entireSSTableInterDcThrottle > ------------------------------------------------------------------------------ > > Key: CASSANDRA-17677 > URL: https://issues.apache.org/jira/browse/CASSANDRA-17677 > Project: Cassandra > Issue Type: Bug > Components: Tool/bulk load > Reporter: Ekaterina Dimitrova > Assignee: Francisco Guerrero > Priority: Normal > Fix For: 4.1-beta, 4.1.x, 4.x > > Time Spent: 8h > Remaining Estimate: 0h > > {{entire_sstable_stream_throughput_outbound and > entire_sstable_inter_dc_stream_throughput_outbound}} were introduced in > CASSANDRA-17065.They were added to the LoaderOptions class but they are not > loaded in BulkLoader as {{throttle}} and {{interDcThrottle are. }}{{As part > of this ticket we need to fix the BulkLoader, also those properties should be > advertised as MiB/s, not megabits/s. This was not changed in CASSANDRA-15234 > for the bulk loader because those are not loaded and those variables in > LoaderOptions are disconnected from the Cassandra config parameters and > unused at the moment. }} > It will be good also to update the doc here - > [https://cassandra.apache.org/doc/latest/cassandra/operating/bulk_loading.html,|https://cassandra.apache.org/doc/latest/cassandra/operating/bulk_loading.html] > {{and add a test that those are loaded properly when used with the > BulkLoader. }} > {{CC [~frankgh] }} -- 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