Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nodetool compact silently ignores the "-s" option #60

Open
nyh opened this issue Jan 13, 2019 · 0 comments
Open

nodetool compact silently ignores the "-s" option #60

nyh opened this issue Jan 13, 2019 · 0 comments
Milestone

Comments

@nyh
Copy link
Contributor

nyh commented Jan 13, 2019

"nodetool compact" has a "-s" option to ask not to compact all the data into one SSTable, but rather split it into one table in each size tier (one table with half the data, one with 1/4, one with 1/8, etc.). We silently ignore this option. We should probably throw an unimplemented exception in this case instead, not silently ignore the user's explicit request.

I think the culprit is in the scylla-jmx project, in src/main/java/org/apache/cassandra/service/StorageService.java, we have:

    @Override
    public void forceKeyspaceCompaction(boolean splitOutput, String keyspaceName, String... tableNames)
            throws IOException, ExecutionException, InterruptedException {
        // "splitOutput" afaik not relevant for scylla (yet?...)
        forceKeyspaceCompaction(keyspaceName, tableNames);
    }

This just ignores the splitOutput option. It could do something like

throw new UnsupportedOperationException("Compact: 'split_output' (-s) option not supported");
@DoronArazii DoronArazii added this to the Backlog milestone May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants