ddupg commented on PR #1835: URL: https://github.com/apache/phoenix/pull/1835#issuecomment-1968517106
> 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 Thanks @stoty for reviewing and opinion, I'll try to fix it. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
