You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@phoenix.apache.org by "ddupg (via GitHub)" <gi...@apache.org> on 2024/02/26 09:26:11 UTC
[PR] PHOENIX-7237 Pherf unit tests PherfTest and ResourceTest are failing [phoenix]
ddupg opened a new pull request, #1835:
URL: https://github.com/apache/phoenix/pull/1835
(no comment)
--
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-unsubscribe@phoenix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] PHOENIX-7237 Pherf unit tests PherfTest and ResourceTest are failing [phoenix]
Posted by "stoty (via GitHub)" <gi...@apache.org>.
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-unsubscribe@phoenix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] PHOENIX-7237 Pherf unit tests PherfTest and ResourceTest are failing [phoenix]
Posted by "ddupg (via GitHub)" <gi...@apache.org>.
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: issues-unsubscribe@phoenix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org