[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. KUDU-1880 Prevent eager ignoring of NULLs During a nullable block scan, if a NULL set of cells is reached

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-28 Thread Andrew Wong (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#11). Change subject: KUDU-1880 Prevent eager ignoring of NULLs

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 11: Code-Review+2 lgtm, assuming tests pass -- To view, visit

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-28 Thread Andrew Wong (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#10). Change subject: KUDU-1880 Prevent eager ignoring of NULLs

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6029 To unsubscribe, visit

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-28 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 9: (9 comments)

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-28 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#9). Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. KUDU-1880 Prevent

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-28 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#8). Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. KUDU-1880 Prevent

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-28 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#7). Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. KUDU-1880 Prevent

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-27 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#6). Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. KUDU-1880 Prevent

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-27 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#5). Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. KUDU-1880 Prevent

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/6029/4/src/kudu/tablet/tablet-scan-correctness-test.cc

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 4: (2 comments)

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6029/4/src/kudu/tablet/tablet-scan-correctness-test.cc

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6029/4/src/kudu/tablet/tablet-scan-correctness-test.cc

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/6029/4/src/kudu/cfile/binary_dict_block.h File

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/6029/3/src/kudu/tablet/tablet-scan-correctness-test.cc

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-23 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#4). Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. KUDU-1880 Prevent

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/6029/3/src/kudu/tablet/tablet-scan-correctness-test.cc

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-23 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#3). Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. KUDU-1880 Prevent

[kudu-CR] KUDU-1880 Prevent eager ignoring of NULLs

2017-02-15 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6029 to look at the new patch set (#2). Change subject: KUDU-1880 Prevent eager ignoring of NULLs .. KUDU-1880 Prevent

[kudu-CR] [KUDU-1880] Prevent eager ignoring of NULLs

2017-02-15 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: [KUDU-1880] Prevent eager ignoring of NULLs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6029/1//COMMIT_MSG Commit Message: PS1, Line 7:

[kudu-CR] [KUDU-1880] Prevent eager ignoring of NULLs

2017-02-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [KUDU-1880] Prevent eager ignoring of NULLs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6029/1//COMMIT_MSG Commit Message: PS1, Line

[kudu-CR] [KUDU-1880] Prevent eager ignoring of NULLs

2017-02-15 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new change for review. http://gerrit.cloudera.org:8080/6029 Change subject: [KUDU-1880] Prevent eager ignoring of NULLs .. [KUDU-1880] Prevent eager ignoring of NULLs While scanning nullable