[
https://issues.apache.org/jira/browse/CASSANDRA-14442?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16497552#comment-16497552
]
Jordan West commented on CASSANDRA-14442:
-----------------------------------------
[~krummas] LGTM overall. One thing I wanted to check: is it ok that callers of
importNewSSTables can now run concurrently with callers of
CFS#runWithCompactionsDisabled (callers like truncate and clearUnsafe)?
Some minor things:
* Remove the whitespace only change in Tracker
* Rename first argument of CFS#importNewSSTables to srcPaths
* Consider moving {{SSTableImporter#moveAndOpenSSTable}} to be a static method
on SSTable, maybe {{renameAndOpen}} (it may be useful for future uses/tests and
isn’t specific to {{SSTableImporter}})
* Thanks for adding the new dtests. Should they be marked since 4.0?
> Let nodetool import take a list of directories
> ----------------------------------------------
>
> Key: CASSANDRA-14442
> URL: https://issues.apache.org/jira/browse/CASSANDRA-14442
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Marcus Eriksson
> Assignee: Marcus Eriksson
> Priority: Major
> Fix For: 4.x
>
>
> It should be possible to load sstables from several input directories when
> running nodetool import. Directories that failed to import should be output.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]