FrankChen021 commented on PR #20152: URL: https://github.com/apache/druid/pull/20152#issuecomment-5452723948
> Not the same — same class of fix, different code path. #20151 covers the broker response path that feeds JsonParserIterator. This one is the second leg: the handleResponse variant of DirectDruidClient, which #20151 doesn't touch. Both are needed; no overlap between the diffs. I think you can put these changes in one PR as these changes are similar and in same class. I'm worrying about repeated code in different paths -- 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]
