risdenk commented on PR #1044: URL: https://github.com/apache/solr/pull/1044#issuecomment-1274710727
Well @madrob thanks for the pointer - weirdly https://github.com/apache/solr/blob/main/solr/packaging/test/test_modules.bats#L35 doesn't fail. I would expect it to :( I'll keep digging. > I suspect maybe it's because the security manager in tests allows access to CWD (maybe?) but not in a real-world scenario. even if CWD is allowed, I'd be surprised if `"../calcite-test-dataset"` or `"../../calcite-test-dataset"` is allowed in tests... I don't know why we don't see this in tests yet. -- 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
