Daniel Becker has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/14029


Change subject: IMPALA-8840: Check failed: num_bytes <= sizeof(T) (5 vs. 4)
......................................................................

IMPALA-8840: Check failed: num_bytes <= sizeof(T) (5 vs. 4)

Added a DCHECK to RleBatchDecoder so that it does not accept bit widths
higher than the width of its type parameter.

Also preventing UnpackAndDecodeValues from using higher bit widths than
32, the width of its dictionary index type, with a static assert.

Change-Id: I93461ba2cabb5ec7e0b65dcd62844fcbfa597d16
---
M be/src/util/bit-packing.h
M be/src/util/bit-packing.inline.h
M be/src/util/dict-test.cc
M be/src/util/rle-encoding.h
4 files changed, 13 insertions(+), 7 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/29/14029/1
--
To view, visit http://gerrit.cloudera.org:8080/14029
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I93461ba2cabb5ec7e0b65dcd62844fcbfa597d16
Gerrit-Change-Number: 14029
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Becker <daniel.bec...@cloudera.com>

Reply via email to