dlmarion commented on PR #4338: URL: https://github.com/apache/accumulo/pull/4338#issuecomment-1979493928
> I removed this code with my changes in #4273 since the config is no longer across multiple properties. But would it make sense to add some sort of on-demand SPI validation utility? https://github.com/apache/accumulo/blob/2.1/server/base/src/main/java/org/apache/accumulo/server/conf/CheckCompactionConfig.java The CompactionPlanner is an SPI, so DefaultCompactionPlanner could implement the validation interface and it would be validated on startup. -- 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. To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
