tolbertam commented on PR #4160: URL: https://github.com/apache/cassandra/pull/4160#issuecomment-2889423809
Did some profiling and the performance delta I was seeing was influenced by the use of `-XX:-TieredCompilation` and `-XX:-BackgroundCompilation` when running tests in my IDE, which was not the case when running `ant testsome -Duse.jdk11=true -Dtest.name=org.apache.cassandra.cql3.PstmtPersistenceTest`.  I've updated `testAsyncPstmtInvalidation` [c85d156](https://github.com/apache/cassandra/pull/3917/commits/c85d1563ee128e6e413956ea4ae31fbf08307196) only load as many statements are necessary to validate the behavior change and it now runs very quickly (~400-800ms) like the other tests do. I'd like to do a little more profiling to understand why things are slower on trunk generally, but I've done enough to rule out these changes as a contributor so I think we can move on. -- 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: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org
