gianm commented on PR #16153: URL: https://github.com/apache/druid/pull/16153#issuecomment-2046146984
> @abhishekagarwal87 @gianm I was just looking at this PR, not sure what you are suggesting for how to make forward progress? Oh, I don't think the discussion about regex libraries and stuff needs to block anything for this particular patch. It was just interesting. I had previously skimmed this patch and the general approach looks OK to me. I think all that was required for making forward progress was a committer to take more than a skim level look. I didn't have bandwidth to do more than that until today, but I can take a look today. -- 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: commits-unsubscr...@druid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org