Jdurham2843 commented on PR #1126: URL: https://github.com/apache/solr/pull/1126#issuecomment-1433915434
Hey! I'll start testing the changes you made, and see if the injection errors are still around. From a first look at the changes, I agree with all of them. Regarding the near-duplicate async processing code: I remember trying to dedupe it, but hit a point where I couldn't see how to do it without creating a mess. I'm glad you found a way to make it work, because it felt kind of wrong to leave that in the codebase. -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
