[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-14 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 Craig Webb changed: What|Removed |Added CC||c.w...@crossflight.com -- You are

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-13 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 Christopher Schultz changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 --- Comment #9 from Christopher Schultz --- Getting down to algorithmic complexity, the TimSort is O(n log n) -- same as merge sort. But again, we don't actually need a completely-sorted array. We just need the oldest N items. If you do a

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 --- Comment #8 from Christopher Schultz --- The only other solution I can see is to scan the map values looking for the "oldest" one and evict a maximum of one per scan. Make sure not to evict anything which has changed since the beginning of

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 --- Comment #7 from Remy Maucherat --- Almost certain Chris is right given what the verification code in the JVM does. Isn't the solution a bit costly maybe ? -- You are receiving this mail because: You are the assignee for the bug.

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 --- Comment #6 from spiffy2...@hotmail.com --- Here are the jdbc interceptor configurations that we are using for the slow query report. I meant to attach them earlier, but I couldn't find them in our code.

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 --- Comment #5 from Christopher Schultz --- An updated patch which avoids one of the race-conditions identified in the previous patch. Comments welcome. Only compile-tested at this point. diff --git

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 Mark Thomas changed: What|Removed |Added Component|Util|jdbc-pool Target Milestone|-

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 --- Comment #3 from Christopher Schultz --- Here is a patch that has been only compile-tested. It introduces a small class to represent the important information used to make decisions during removeOldest() and creates a (fairly) stable

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 --- Comment #2 from Christopher Schultz --- Which version of tomcat-pool is in use? I'll bet the problem is that the QueryStats are changing while the slow-query report is running. SlowQueryReport.java:222 is a call to Collections.sort()

[Bug 64415] Error in slow query report - Comparison method violates its general contract!

2020-05-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64415 --- Comment #1 from Michael Osipov --- I have seen this contract violations once in a while too, but where able to reproduce it. -- You are receiving this mail because: You are the assignee for the bug.