Fwiw, I had already started addressing some of those comments, so I plan to raise a follow up PR for that.
Btw, @leventov what did you intend by doing an approval + also leaving comments? I'd interpret that as "please consider these comments, but if you don't want to do them, I am ok with that." I thought they were all reasonable comments so that's why I'm doing a follow up. [ Full content available at: https://github.com/apache/incubator-druid/pull/6220 ] This message was relayed via gitbox.apache.org for [email protected]
