capistrant commented on issue #9224: Create new dynamic config to pause coordinator helpers when needed URL: https://github.com/apache/druid/pull/9224#issuecomment-576853727 Upon some further inspection. I'm not really sure what a good way to test the functionality would be. The unit tests cover serde of the dynamic config. So we should catch any regression issues with the actual configuration via testing in the future. The use of the config item is super elementary. If the boolean is set, it is set, and the conditional will not return true, thus the helpers will be skipped. And whether or not the boolean is set in the config object should be covered by the config unit tests, should it not. So maybe there is not a test to be written. Unless we wanted to write an integration test that confirms that if you run an indexing job while coordinator is paused, even though the job completes, you can poll and see the segment doesn't load for x time. and then enable coordinator and confirm segment loads. But you don't really know if you tested the feature there or if the stars aligned and the timing just happened to work out... Or maybe a test that asserts something was logged, but I don't like lines that rely on logs. very fragile.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
