scylladb / scylla-jmx

Scylla JMX proxy
GNU Affero General Public License v3.0
29 stars 54 forks source link

wire up StorageServiceMean.forceTerminateAllRepairSessions #47

Closed slivne closed 6 years ago

slivne commented 7 years ago

scylladb/scylla@a59e375aadb0ab42eac8f69003cafa868cf9d99a implements the ability to terminate repair session - lets wire it up

amnonh commented 6 years ago

It's already wire up, the jmx supported it all along but the API used to return not implemented, @asias patch connected the API which was the missing part.

Nothing todo. @slivne this can be closed

slivne commented 6 years ago

did you test it ?

On Sun, Feb 11, 2018 at 6:04 PM, Amnon Heiman notifications@github.com wrote:

It's already wire up, the jmx supported it all along but the API used to return not implemented, @asias https://github.com/asias patch connected the API which was the missing part.

Nothing todo. @slivne https://github.com/slivne this can be closed

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/scylladb/scylla-jmx/issues/47#issuecomment-364762428, or mute the thread https://github.com/notifications/unsubscribe-auth/ADThCLvGFbCEq4RIuY0lC7jLhdoRZ79Bks5tTw94gaJpZM4PVzr3 .

amnonh commented 6 years ago

I tested that it is wired, didn't test the repair mechanism.

I connect with jconsole and run the method of the mbean, and got in the traces: [shard 0] repair - Aborted 0 repair job(s)