[kudu-CR] WIP: [iwyu] first pass

2016-11-29 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4738 to look at the new patch set (#7). Change subject: WIP: [iwyu] first pass .. WIP: [iwyu] first pass Updated C++

[kudu-CR] KUDU-1757: fix appendCellValueDebugString, do not throw exception when a column is not set

2016-11-29 Thread YanlongZheng (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5237 to look at the new patch set (#2). Change subject: KUDU-1757: fix appendCellValueDebugString, do not throw exception when a column is not set

[kudu-CR] WIP: KUDU-1767. Create a client flush interleave test

2016-11-29 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: WIP: KUDU-1767. Create a client flush interleave test .. Patch Set 2: Thanks for the review, Alexey. Yes, this test is currently pretty messy. I do need to clean

[kudu-CR] KUDU-798 (part 1) Unify leader/follower mvcc behavior

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-798 (part 1) Unify leader/follower mvcc behavior .. Patch Set 12: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5055 To unsubscribe, visit

[kudu-CR] encodings: change IsBlockFull() to not take a limit parameter

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: encodings: change IsBlockFull() to not take a limit parameter .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5200 To unsubscribe,

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: bshuf_block: some code cleanup .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5193 To unsubscribe, visit

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: bshuf_block: some code cleanup .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5193/6/src/kudu/cfile/bshuf_block.cc File

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: bshuf_block: some code cleanup .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5193 To unsubscribe, visit

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5193 to look at the new patch set (#7). Change subject: bshuf_block: some code cleanup .. bshuf_block:

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: bshuf_block: some code cleanup .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5193/6/src/kudu/cfile/bshuf_block.cc File

[kudu-CR] KUDU-1766: Java client partition pruning with MAX VALUE equality predicate fails

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1766: Java client partition pruning with MAX_VALUE equality predicate fails .. Patch Set 1: (1 comment) Just passing through...

[kudu-CR] bshuf block: fix GetFirstKey() and GetLastKey()

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: bshuf_block: fix GetFirstKey() and GetLastKey() .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5192 To unsubscribe, visit

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 12: (15 comments) http://gerrit.cloudera.org:8080/#/c/4996/12/src/kudu/integration-tests/fuzz-itest.cc File

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4995 to look at the new patch set (#22). Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files ..

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files .. Patch Set 12: (1 comment)

[kudu-CR] encodings: change IsBlockFull() to not take a limit parameter

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: encodings: change IsBlockFull() to not take a limit parameter .. encodings: change IsBlockFull() to not take a limit parameter Everywhere we called this,

[kudu-CR] WIP: Support safe time advancement on replicas in the absense of writes

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: Support safe time advancement on replicas in the absense of writes .. Patch Set 3: (4 comments)

[kudu-CR] [i-tests] test for timestamp propagation with write ops

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5268 Change subject: [i-tests] test for timestamp propagation with write ops .. [i-tests] test for timestamp propagation with write ops

[kudu-CR] bshuf block: some low-hanging-fruit optimizations on write path

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: bshuf_block: some low-hanging-fruit optimizations on write path .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/5195/5//COMMIT_MSG Commit

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files .. Patch Set 22: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4995 To unsubscribe,

[kudu-CR] [i-tests] scan token timestamp propagation test

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [i-tests] scan token timestamp propagation test .. Patch Set 1: (1 comment)

[kudu-CR] Don't output unobservable rows from the MemRowset

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4994 to look at the new patch set (#15). Change subject: Don't output unobservable rows from the MemRowset .. Don't

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4995 to look at the new patch set (#20). Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files ..

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files .. Patch Set 20: (6 comments)

[kudu-CR] KUDU-1634 (part 3). Changed tmp files' infix from ".tmp" to ".kudutmp"

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1634 (part 3). Changed tmp files' infix from ".tmp" to ".kudutmp" .. Patch Set 3: Hey Maxim, looks like this patch has some conflicts and needs to be

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4996 to look at the new patch set (#14). Change subject: Add snapshot scans to fuzz-itest .. Add snapshot scans to

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4995 to look at the new patch set (#21). Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files ..

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files .. Patch Set 20: (5 comments)

[kudu-CR] KUDU-1766: Java client partition pruning with MAX VALUE equality predicate fails

2016-11-29 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5266 to review the following change. Change subject: KUDU-1766: Java client partition pruning with MAX_VALUE equality predicate fails

[kudu-CR] KUDU-1766: Java client partition pruning with MAX VALUE equality predicate fails

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1766: Java client partition pruning with MAX_VALUE equality predicate fails .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] WIP: Support safe time advancement on replicas in the absense of writes

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5240 to look at the new patch set (#4). Change subject: WIP: Support safe time advancement on replicas in the absense of writes

[kudu-CR] bshuf block: fix GetFirstKey() and GetLastKey()

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: bshuf_block: fix GetFirstKey() and GetLastKey() .. bshuf_block: fix GetFirstKey() and GetLastKey() This fixes a bug where the bitshuffle encoding would

[kudu-CR] cfile-test: use a faster data generator for 100M-string test

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: cfile-test: use a faster data generator for 100M-string test .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5201 To unsubscribe,

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4995 to look at the new patch set (#23). Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: bshuf_block: some code cleanup .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5193/6/src/kudu/cfile/bshuf_block.cc File

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: bshuf_block: some code cleanup .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5193/6/src/kudu/cfile/bshuf_block.cc File

[kudu-CR] KUDU-1766: Java client partition pruning with MAX VALUE equality predicate fails

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1766: Java client partition pruning with MAX_VALUE equality predicate fails .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1766: Java client partition pruning with MAX VALUE equality predicate fails

2016-11-29 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5266 to look at the new patch set (#2). Change subject: KUDU-1766: Java client partition pruning with MAX_VALUE equality predicate fails

[kudu-CR] [c++ client] propagate timestamp for write operations

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5269 Change subject: [c++ client] propagate timestamp for write operations .. [c++ client] propagate timestamp for write operations Updated

[kudu-CR] bshuf block: some low-hanging-fruit optimizations on write path

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: bshuf_block: some low-hanging-fruit optimizations on write path .. Patch Set 5: (1 comment)

[kudu-CR] Update debug partition and row printing

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Update debug partition and row printing .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/5262/1/src/kudu/common/partition.cc File

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files .. Patch Set 19: (10 comments)

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: bshuf_block: some code cleanup .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5193/6/src/kudu/cfile/bshuf_block.cc File

[kudu-CR] Don't output unobservable rows from the MemRowset

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4994 to look at the new patch set (#14). Change subject: Don't output unobservable rows from the MemRowset .. Don't

[kudu-CR] encodings: change IsBlockFull() to not take a limit parameter

2016-11-29 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5200 to look at the new patch set (#3). Change subject: encodings: change IsBlockFull() to not take a limit parameter

[kudu-CR] WIP: KUDU-1767. Create a client flush interleave test

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: WIP: KUDU-1767. Create a client flush interleave test .. Patch Set 2: (7 comments) It looks like a nice test to isolate this problem!

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: bshuf_block: some code cleanup .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/5193/3/src/kudu/cfile/bshuf_block.cc File

[kudu-CR] Update debug partition and row printing

2016-11-29 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Dimitris Tsirogiannis, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5262 to review the following change. Change subject: Update debug partition and row printing

[kudu-CR] bshuf block: some code cleanup

2016-11-29 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5193 to look at the new patch set (#6). Change subject: bshuf_block: some code cleanup .. bshuf_block: some code

[kudu-CR] tablet copy: Rename tablet copy session source files

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tablet copy: Rename tablet copy session source files .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5042 To unsubscribe, visit

[kudu-CR] tablet copy: Rename tablet copy session source files

2016-11-29 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: tablet copy: Rename tablet copy session source files .. tablet copy: Rename tablet copy session source files This is just a quick cleanup to the class

[kudu-CR] tablet copy: Fix tidy warnings in TabletCopySourceSession{Test}

2016-11-29 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: tablet copy: Fix tidy warnings in TabletCopySourceSession{Test} .. tablet copy: Fix tidy warnings in TabletCopySourceSession{Test} Done as a follow-up

[kudu-CR] [i-tests] scan token timestamp propagation test

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [i-tests] scan token timestamp propagation test .. Patch Set 1: (2 comments)

[kudu-CR] Update debug partition and row printing

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Update debug partition and row printing .. Patch Set 1: screenshots of updated web UIs: http://imgur.com/a/VNlMA -- To view, visit

[kudu-CR] cmake: Throw cmake error if unit test does not exist

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: cmake: Throw cmake error if unit test does not exist .. cmake: Throw cmake error if unit test does not exist Previously, if the unit test file did not

[kudu-CR] cmake: Throw cmake error if unit test does not exist

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: cmake: Throw cmake error if unit test does not exist .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5222 To unsubscribe, visit

[kudu-CR] cfile set-test: switch to using INT32 instead of UINT32

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: cfile_set-test: switch to using INT32 instead of UINT32 .. cfile_set-test: switch to using INT32 instead of UINT32 We don't actually support users using

[kudu-CR] encodings: change IsBlockFull() to not take a limit parameter

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: encodings: change IsBlockFull() to not take a limit parameter .. Patch Set 2: (5 comments)

[kudu-CR] [i-tests] scan token timestamp propagation test

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [i-tests] scan token timestamp propagation test .. Patch Set 1: (2 comments)

[kudu-CR] Don't output unobservable rows from the MemRowset

2016-11-29 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Don't output unobservable rows from the MemRowset .. Patch Set 13: > > DRS 1: > undo: del @ 2 > base: v1 > redo delta: del @3 > > DRS 2: > base: v2 > >

[kudu-CR] KUDU-798 (part 2) Remove automatic safe time adjustment from mvcc

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-798 (part 2) Remove automatic safe time adjustment from mvcc .. Patch Set 13: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5057 To

[kudu-CR] KUDU-1757: fix appendCellValueDebugString, do not throw exception when a column is not set

2016-11-29 Thread YanlongZheng (Code Review)
YanlongZheng has posted comments on this change. Change subject: KUDU-1757: fix appendCellValueDebugString, do not throw exception when a column is not set .. Patch Set 2: (2 comments)

[kudu-CR] Add Reinserts to tablet history gc-itest

2016-11-29 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add Reinserts to tablet_history_gc-itest .. Patch Set 13: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4997 To unsubscribe, visit

[kudu-CR] [c++] ColumnPredicate simplification for inequalities on boundary values

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [c++] ColumnPredicate simplification for inequalities on boundary values .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5273/1//COMMIT_MSG

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4996/12/src/kudu/integration-tests/fuzz-itest.cc File

[kudu-CR] WIP: Support safe time advancement on replicas in the absense of writes

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5240 to look at the new patch set (#7). Change subject: WIP: Support safe time advancement on replicas in the absense of writes

[kudu-CR] Update debug partition and row printing

2016-11-29 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5262 to look at the new patch set (#2). Change subject: Update debug partition and row printing .. Update debug

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files .. Patch Set 24: Code-Review+2 just a rebase keeping todd's +2 -- To view, visit

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4996/12/src/kudu/integration-tests/fuzz-itest.cc File

[kudu-CR] WIP: Support safe time advancement on replicas in the absense of writes

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5240 to look at the new patch set (#6). Change subject: WIP: Support safe time advancement on replicas in the absense of writes

[kudu-CR] Don't output unobservable rows from the MemRowset

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Don't output unobservable rows from the MemRowset .. Patch Set 17: Code-Review+1 (5 comments) just a rebase, keeping todd's +1

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4996/12/src/kudu/integration-tests/fuzz-itest.cc File

[kudu-CR] [java-client] KuduPredicate simplification for inequalities on boundary values

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java-client] KuduPredicate simplification for inequalities on boundary values .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5272/1//COMMIT_MSG

[kudu-CR] [c++] ColumnPredicate simplification for inequalities on boundary values

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [c++] ColumnPredicate simplification for inequalities on boundary values .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5273 To

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4996/12/src/kudu/integration-tests/fuzz-itest.cc File

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 16: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4996 To unsubscribe, visit

[kudu-CR] cfile: make encoder/decoder classes final

2016-11-29 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5202 to look at the new patch set (#4). Change subject: cfile: make encoder/decoder classes final ..

[kudu-CR] Remove GROUP VARINT encoding

2016-11-29 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5271 to review the following change. Change subject: Remove GROUP_VARINT encoding .. Remove GROUP_VARINT encoding This

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4996/12/src/kudu/integration-tests/fuzz-itest.cc File

[kudu-CR] binary plain block: avoid a virtual call in hot path

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: binary_plain_block: avoid a virtual call in hot path .. Patch Set 3: Code-Review-1 Dan and I actaully just chatted, and it sounds like a better solution with

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4996/12/src/kudu/integration-tests/fuzz-itest.cc File

[kudu-CR] Don't output unobservable rows from the MemRowset

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4994 to look at the new patch set (#16). Change subject: Don't output unobservable rows from the MemRowset

[kudu-CR] [i-tests] scan token timestamp propagation test

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [i-tests] scan token timestamp propagation test .. Patch Set 1: (1 comment)

[kudu-CR] Add Reinserts to tablet history gc-itest

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add Reinserts to tablet_history_gc-itest .. Patch Set 16: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4997 To unsubscribe, visit

[kudu-CR] cfile-test: use a faster data generator for 100M-string test

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: cfile-test: use a faster data generator for 100M-string test .. Patch Set 3: I don't think it'll be that much less effective, since now it'll at least have the

[kudu-CR] Don't output unobservable rows from the MemRowset

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Don't output unobservable rows from the MemRowset .. Patch Set 15: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4994 To unsubscribe, visit

[kudu-CR] WIP: Support safe time advancement on replicas in the absense of writes

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5240 to look at the new patch set (#5). Change subject: WIP: Support safe time advancement on replicas in the absense of writes

[kudu-CR] cfile-test: use a faster data generator for 100M-string test

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: cfile-test: use a faster data generator for 100M-string test .. Patch Set 3: This is going to make the prefix encoder less effective on the dataset, is that

[kudu-CR] cfile-test: use a faster data generator for 100M-string test

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: cfile-test: use a faster data generator for 100M-string test .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5201 To unsubscribe,

[kudu-CR] Remove GROUP VARINT encoding

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Remove GROUP_VARINT encoding .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5271 To unsubscribe, visit

[kudu-CR] KUDU-1766: Java client partition pruning with MAX VALUE equality predicate fails

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1766: Java client partition pruning with MAX_VALUE equality predicate fails .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] cfile-test: use a faster data generator for 100M-string test

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: cfile-test: use a faster data generator for 100M-string test .. Patch Set 3: Right, forgot about the padding. -- To view, visit

[kudu-CR] cfile-test: use a faster data generator for 100M-string test

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: cfile-test: use a faster data generator for 100M-string test .. cfile-test: use a faster data generator for 100M-string test The current data generator

[kudu-CR] bshuf block: some low-hanging-fruit optimizations on write path

2016-11-29 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: bshuf_block: some low-hanging-fruit optimizations on write path .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/5195/5//COMMIT_MSG Commit

[kudu-CR] cfile: make encoder/decoder classes final

2016-11-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: cfile: make encoder/decoder classes final .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5202 To unsubscribe, visit

[kudu-CR] [i-tests] scan token timestamp propagation test

2016-11-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [i-tests] scan token timestamp propagation test .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1766: Java client partition pruning with MAX VALUE equality predicate fails

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: KUDU-1766: Java client partition pruning with MAX_VALUE equality predicate fails .. KUDU-1766: Java client partition pruning with MAX_VALUE equality

[kudu-CR] KUDU-1634 (part 3). Changed tmp files' infix from ".tmp" to ".kudutmp"

2016-11-29 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5123 to look at the new patch set (#4). Change subject: KUDU-1634 (part 3). Changed tmp files' infix from ".tmp" to ".kudutmp"

[kudu-CR] KUDU-1634 (part 3). Changed tmp files' infix from ".tmp" to ".kudutmp"

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1634 (part 3). Changed tmp files' infix from ".tmp" to ".kudutmp" .. Patch Set 4: > Hey Maxim, looks like this patch has some conflicts and needs to be >

[kudu-CR] bshuf block: some low-hanging-fruit optimizations on write path

2016-11-29 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: bshuf_block: some low-hanging-fruit optimizations on write path .. bshuf_block: some low-hanging-fruit optimizations on write path Rather than adding an

  1   2   >