You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jmeter.apache.org by GitBox <gi...@apache.org> on 2022/11/02 20:17:20 UTC

[GitHub] [jmeter] FSchumacher commented on pull request #693: ci: add randomized matrix for better test coverage

FSchumacher commented on PR #693:
URL: https://github.com/apache/jmeter/pull/693#issuecomment-1301178336

   Looks good (as always :)) I have a few questions, though.
   * Has the usage of the synchronized identitymap a major performance impact and should we try to get it back to something concurrent?
   * Is the copyright by you line in  `matrix_build.js` file intentionally?


-- 
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: dev-unsubscribe@jmeter.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org