This is an automated email from the ASF dual-hosted git repository.

wyk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git


The following commit(s) were added to refs/heads/master by this push:
     new 8c7d28d769 [ASTERIXDB-3161][STO] Avoid comparing lengths' bytes for 
strings
8c7d28d769 is described below

commit 8c7d28d7695e927a34e21f0f7e41b463e57a2380
Author: Wail Alkowaileet <[email protected]>
AuthorDate: Mon Apr 10 16:06:22 2023 -0700

    [ASTERIXDB-3161][STO] Avoid comparing lengths' bytes for strings
    
    - user model changes: no
    - storage format changes: no
    - interface changes: no
    
    Details:
    Avoid comparing the lengths' bytes when
    UTF8StringPointable#compare(IValueReference, IValueReference)
    is called
    
    Change-Id: If7db4b3a09a3abbf3cd946e91e1971d7957d99ef
    Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17484
    Integration-Tests: Jenkins <[email protected]>
    Reviewed-by: Murtadha Hubail <[email protected]>
    Tested-by: Jenkins <[email protected]>
---
 .../org/apache/hyracks/data/std/primitive/UTF8StringPointable.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/hyracks-fullstack/hyracks/hyracks-data/hyracks-data-std/src/main/java/org/apache/hyracks/data/std/primitive/UTF8StringPointable.java
 
b/hyracks-fullstack/hyracks/hyracks-data/hyracks-data-std/src/main/java/org/apache/hyracks/data/std/primitive/UTF8StringPointable.java
index 8013e05fc0..f54741d2bf 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-data/hyracks-data-std/src/main/java/org/apache/hyracks/data/std/primitive/UTF8StringPointable.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-data/hyracks-data-std/src/main/java/org/apache/hyracks/data/std/primitive/UTF8StringPointable.java
@@ -219,8 +219,8 @@ public final class UTF8StringPointable extends 
AbstractPointable implements IHas
     }
 
     public static int compare(IValueReference valueA, IValueReference valueB) {
-        return UTF8StringUtil.compareTo(valueA.getByteArray(), 
valueA.getStartOffset(), valueA.getLength(),
-                valueB.getByteArray(), valueB.getStartOffset(), 
valueB.getLength());
+        return UTF8StringUtil.compareTo(valueA.getByteArray(), 
valueA.getStartOffset(), valueB.getByteArray(),
+                valueB.getStartOffset());
     }
 
     /**

Reply via email to