rahulgoswami commented on PR #3347: URL: https://github.com/apache/solr/pull/3347#issuecomment-2887202955
Thanks for taking a look Eric. I converted the PR to a draft since I am still polishing off the tests. It is currently failing, but I committed it anyway in order to be able to pull it on macOS to see if the test fails for Linux too (it does!). The bug is in the test, not the fix, and it is seems to be because of a nuance in the /admin/info/properties that got inadvertently uncovered as part of this testing. I am fairly certain this would work if I retrieve the system property from the Java Runtime of the launched instance instead of calling /admin/info/properties. Now that I know the reason, I am a little more at peace and will revisit this later 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: 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
