Moti Asayag has submitted this change and it was merged. Change subject: core: Fixed lexo-numeric comparator for big numbers ......................................................................
core: Fixed lexo-numeric comparator for big numbers The lexo-numeric comparator was using Integer class which is very limited to parse numbers, and would cause an exception if the sequence of numbers if too long. Change to BigInteger which is unlimited in size, and has the required functionality. There doesn't seem to be any adverse effect on the performance of the comparison. Change-Id: I52ec8c73e85b2505314a4043206d5883e2a43e37 Bug-Url: https://bugzilla.redhat.com/986979 Signed-off-by: Mike Kolesnik <[email protected]> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/comparators/LexoNumericComparator.java M backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/businessentities/comparators/LexoNumericComparatorTest.java 2 files changed, 8 insertions(+), 2 deletions(-) Approvals: Mike Kolesnik: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17238 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52ec8c73e85b2505314a4043206d5883e2a43e37 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <[email protected]> Gerrit-Reviewer: Lior Vernia <[email protected]> Gerrit-Reviewer: Mike Kolesnik <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
