stoty commented on PR #1835: URL: https://github.com/apache/phoenix/pull/1835#issuecomment-1968288626
I don't like this, as this is very brittle, and we will have the same problem again soon. (The previous patch has the same problem) The real problem is that pherf seems to try to load its test config from anywhere on the classpath. I would prefer that the logic in getResourcesPath is fixed so that it does not recursively search the whole classpath. https://github.com/apache/phoenix/blob/bc1e2e7bea40c7d03940748e8f1d9f6b23339867/phoenix-pherf/src/main/java/org/apache/phoenix/pherf/util/ResourceList.java#L103 -- 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...@phoenix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
