search

apache / accumulo

Apache Accumulo
https://accumulo.apache.org
Apache License 2.0
1.08k stars 445 forks source link

Cleanup InstanceOperations API for active scans/compactions #5102

Open cshannon opened 4 days ago

cshannon commented 4 days ago

This simplifies the API for getting active scans and active compactions. The new API has been updated to just use a collection of servers instead of trying to also support other arguments (like a single server). Other methods were removed if new or deprecated if old, such as getActiveCompactions().

Getting active scans was updated to be multi-threaded like getting active compactions, and they now share the same code. If there are not multiple servers to query then the code now just uses a direct executor to prevent creating a thread pool.

The listscans command was updated to take advantage of the new API and to use multiple threads to collect the scans to speed things up.

Lastly, the tests have been updated to account for the API changes.

This closes #5096

cshannon commented 4 days ago

@keith-turner - this is ready for another review after the latest updates