[kudu] branch master updated: bitshuffle: check for a multiple-of-8-bytes invariant in Finish

[ad...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 0bc19d2  bitshuffle: check for a multiple-of-8-bytes invariant in Finish
0bc19d2 is described below

commit 0bc19d2df62efe60ecb7bd7bd5e63cc210a17550
Author: Adar Dembo <ad...@cloudera.com>
AuthorDate: Sat Apr 4 14:38:36 2020 -0700

    bitshuffle: check for a multiple-of-8-bytes invariant in Finish
    
    Change-Id: I447a78739f19ee5a7dfd8935bdfb273a5b71369a
    Reviewed-on: http://gerrit.cloudera.org:8080/15652
    Reviewed-by: Bankim Bhavsar <ba...@cloudera.com>
    Reviewed-by: Andrew Wong <aw...@cloudera.com>
    Tested-by: Kudu Jenkins
---
 src/kudu/cfile/bshuf_block.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/kudu/cfile/bshuf_block.h b/src/kudu/cfile/bshuf_block.h
index 52e8639..60c1545 100644
--- a/src/kudu/cfile/bshuf_block.h
+++ b/src/kudu/cfile/bshuf_block.h
@@ -183,6 +183,7 @@ class BShufBlockBuilder final : public BlockBuilder {
     for (int i = 0; i < padding_bytes; i++) {
       data_.push_back(0);
     }
+    DCHECK_EQ(0, data_.length() % 8);
 
     buffer_.resize(kHeaderSize +
                    bitshuffle::compress_lz4_bound(num_elems_after_padding, final_size_of_type, 0));