vlsi commented on PR #693: URL: https://github.com/apache/jmeter/pull/693#issuecomment-1301295153
>synchronized identitymap a major performance impact and should we try to get it back to something concurrent? I've no idea. We already had synchronized maps, so it should not be much worse. An alternative option would be changing AbstractTestElement.equals to this==that conparison, and add contentEquals(AbstractTestElement other) method. It might be somewhat surprising if clients used equals, however, then we won't need to use IdentityHashMap all over the place. WDYT? -- 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-unsubscr...@jmeter.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
