Yingyi Bu has submitted this change and it was merged.

Change subject: ASTERIXDB-1334 and ASTERIXDB-1616: Fix null bitmap size.
......................................................................


ASTERIXDB-1334 and ASTERIXDB-1616: Fix null bitmap size.

Change-Id: I1eaa215b8fc2df02fecb7f86cc5169f07765b09d
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1163
Sonar-Qube: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>
Integration-Tests: Jenkins <[email protected]>
Reviewed-by: Yingyi Bu <[email protected]>
---
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/builders/RecordBuilder.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Yingyi Bu: Looks good to me, approved
  Jenkins: Verified; No violations found; Verified



diff --git 
a/asterixdb/asterix-om/src/main/java/org/apache/asterix/builders/RecordBuilder.java
 
b/asterixdb/asterix-om/src/main/java/org/apache/asterix/builders/RecordBuilder.java
index 7e6d97e..ca7c0b0 100644
--- 
a/asterixdb/asterix-om/src/main/java/org/apache/asterix/builders/RecordBuilder.java
+++ 
b/asterixdb/asterix-om/src/main/java/org/apache/asterix/builders/RecordBuilder.java
@@ -157,7 +157,7 @@
                 // the null bit is 1, which means the field is not a null,
                 // the missing bit is 0, means the field is missing (by 
default).
                 Arrays.fill(nullBitMap, 0, nullBitMapSize, (byte) 0xAA);
-                headerSize += nullBitMap.length;
+                headerSize += nullBitMapSize;
             }
         }
     }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1eaa215b8fc2df02fecb7f86cc5169f07765b09d
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Yingyi Bu <[email protected]>

Reply via email to