fectrain commented on PR #15706: URL: https://github.com/apache/druid/pull/15706#issuecomment-1897897185
> @fectrain , there seem to be a few code coverage issues. Please see if you can address them by adding /updating unit tests. Hi, @kfaraz, there seems no a easy way to resolve this coverage issue, we might cover them by enable debug level log for the whole unit test package by modifying the log property files, but it seems not necessary? As my change is just adding a condition check and an indentation for the original log line, which should not able to modify the original logic or bring in exceptions, so I think we can merge this change. WDYT? -- 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]
