Utils.sortTopN does not handle single-element arrays

kashipai / h2database

H2 Database for reference.

0 stars 0 forks source link

Utils.sortTopN does not handle single-element arrays #439

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

This caused IllegalArgumentException in Utils.partialQuickSort when picking a 
random pivot.  I noticed this specifically in queries of the form: "SELECT 
column FROM table ORDER BY column".  Fix and unit test attached.

Original issue reported on code.google.com by g...@maginatics.com on 9 Feb 2013 at 11:11

Attachments:

h2_partial_sort_one_element.patch

GoogleCodeExporter commented 9 years ago

Thanks a lot! Yes, that's a bug, I will fix it.

Original comment by thomas.t...@gmail.com on 10 Feb 2013 at 3:00

Changed state: Accepted

GoogleCodeExporter commented 9 years ago

Should be fixed in H2 version 1.3.171

Original comment by thomas.t...@gmail.com on 17 Mar 2013 at 4:09

Changed state: Fixed