clintropolis opened a new pull request, #19206: URL: https://github.com/apache/druid/pull/19206
More follow-up to #19109, #19166, and #19173, this PR removes the rest of the `InputRowParser` implementations since they were effectively only used in tests at this point. Will do a pass on `ParseSpec` is next, since I think all of those can go away too, but need to do some more test refactoring and this PR seemed more than big enough already. I modified `AggregationTestHelper` to just use code versions of input schema and format since imo it seems very tedious to put json specs in strings and use that, but as a side-effect this PR needs a follow-up to drop using strings to define the aggregator factories and queries to make them consistent. -- 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]
