Taewoo Kim has submitted this change and it was merged.

Change subject: Optimized the binary tokenizer - get the total number of tokens 
Change-Id: I83765b6cd7cb92648aa55b260b0044fa3ffdcd7d Reviewed-on: 
https://asterix-gerrit.ics.uci.edu/349 Reviewed-by: Young-Seok Kim 
<[email protected]> Tested-by: Jenkins <jenkins@fulliautom
......................................................................


Optimized the binary tokenizer - get the total number of tokens
Change-Id: I83765b6cd7cb92648aa55b260b0044fa3ffdcd7d
Reviewed-on: https://asterix-gerrit.ics.uci.edu/349
Reviewed-by: Young-Seok Kim <[email protected]>
Tested-by: Jenkins <[email protected]>
---
M 
asterix-om/src/main/java/edu/uci/ics/asterix/dataflow/data/common/AOrderedListBinaryTokenizer.java
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  Young-Seok Kim: Looks good to me, approved
  Jenkins: Verified



diff --git 
a/asterix-om/src/main/java/edu/uci/ics/asterix/dataflow/data/common/AOrderedListBinaryTokenizer.java
 
b/asterix-om/src/main/java/edu/uci/ics/asterix/dataflow/data/common/AOrderedListBinaryTokenizer.java
index 7c99b2f..3323fc7 100644
--- 
a/asterix-om/src/main/java/edu/uci/ics/asterix/dataflow/data/common/AOrderedListBinaryTokenizer.java
+++ 
b/asterix-om/src/main/java/edu/uci/ics/asterix/dataflow/data/common/AOrderedListBinaryTokenizer.java
@@ -80,4 +80,9 @@
     protected int getNumberOfItems(byte[] data, int start) {
         return AOrderedListSerializerDeserializer.getNumberOfItems(data, 
start);
     }
+
+    @Override
+    public short getTokensCount() {
+        return (short) listLength;
+    }
 }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/349
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I83765b6cd7cb92648aa55b260b0044fa3ffdcd7d
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Taewoo Kim <[email protected]>
Gerrit-Reviewer: Young-Seok Kim <[email protected]>

Reply via email to