dweiss commented on PR #15490: URL: https://github.com/apache/lucene/pull/15490#issuecomment-3633897187
This is a bit awkward with codecs not permitted to have some kind of attached arguments, I agree. I feel more comfortable with moving the initialization under AssertingCodec's constructor and have it rely on the presence of the top-level target class from the current randomized context. I allowed myself to push it to your branch - please take a look. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
