[kudu-CR] [c++ client] timestamp propagation via scan tokens

2016-11-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] timestamp propagation via scan tokens .. Patch Set 1: I meant making sure that we're not breaking anything with this change stand-alone. We

[kudu-CR] [c++ client] timestamp propagation via scan tokens

2016-11-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++ client] timestamp propagation via scan tokens .. Patch Set 1: > did you test that these tokens are cross-client compatible? Nope, that I didn't do. I

[kudu-CR] [c++ client] timestamp propagation via scan tokens

2016-11-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] timestamp propagation via scan tokens .. Patch Set 1: did you test that these tokens are cross-client compatible? -- To view, visit

[kudu-CR] [java] Reuse snapshot scan timestamp across tablets

2016-11-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [java] Reuse snapshot scan timestamp across tablets .. Patch Set 6: (1 comment)

[kudu-CR] [java] Reuse snapshot scan timestamp across tablets

2016-11-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [java] Reuse snapshot scan timestamp across tablets .. Patch Set 6: (1 comment)

[kudu-CR] [c++ client] timestamp propagation via scan tokens

2016-11-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5220 Change subject: [c++ client] timestamp propagation via scan tokens .. [c++ client] timestamp propagation via scan tokens Implemented

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

2016-11-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5219 Change subject: [i-tests] scan token timestamp propagation test .. [i-tests] scan token timestamp propagation test Added an integration

[kudu-CR] [java] Reuse snapshot scan timestamp across tablets

2016-11-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [java] Reuse snapshot scan timestamp across tablets .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5188 To unsubscribe,

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

2016-11-24 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 (#16). 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-24 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 (#15). 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-24 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 14: (5 comments)

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

2016-11-24 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 14: (5 comments)

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

2016-11-24 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5042 to look at the new patch set (#2). Change subject: tablet copy: Rename tablet copy session source files .. tablet

[kudu-CR] tablet copy: Rename TabletCopySession -> TabletCopySourceSession

2016-11-24 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: tablet copy: Rename TabletCopySession -> TabletCopySourceSession .. tablet copy: Rename TabletCopySession -> TabletCopySourceSession This opens the way

[kudu-CR] [WIP][spark] - Java Spark API DONT BUILD

2016-11-24 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5172 to look at the new patch set (#3). Change subject: [WIP][spark] - Java Spark API DONT_BUILD .. [WIP][spark] - Java

[kudu-CR] For the first time of building bitshuffle, `rm bitshuffle.a` will return an exit code 1 and the build will fail because "-e" option is set in build-thirdparty.sh. bitshuffle.a should be remo

2016-11-24 Thread Kousuke Saruta (Code Review)
Kousuke Saruta has abandoned this change. Change subject: For the first time of building bitshuffle, `rm bitshuffle.a` will return an exit code 1 and the build will fail because "-e" option is set in build-thirdparty.sh. bitshuffle.a should be removed only when it exists.

[kudu-CR] Note to install "which" command in the installation guide in case of building on RHEL/CentOS. I tried to build on Docker container with official CentOS7 image and when I tried to build third

2016-11-24 Thread Kousuke Saruta (Code Review)
Kousuke Saruta has uploaded a new change for review. http://gerrit.cloudera.org:8080/5216 Change subject: Note to install "which" command in the installation guide in case of building on RHEL/CentOS. I tried to build on Docker container with official CentOS7 image and when I tried to build

[kudu-CR] KUDU-1643 Prune hash partitions based on IN-list predicates

2016-11-24 Thread Haijie Hong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5176 to look at the new patch set (#7). Change subject: KUDU-1643 Prune hash partitions based on IN-list predicates ..

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

2016-11-24 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 (#14). Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files ..

[kudu-CR] Make DebugDumpCompactionInput use CompactionInputRowToString

2016-11-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5215 Change subject: Make DebugDumpCompactionInput use CompactionInputRowToString .. Make DebugDumpCompactionInput use

[kudu-CR] KUDU-1643 Prune hash partitions based on IN-list predicates B

2016-11-24 Thread Haijie Hong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5176 to look at the new patch set (#6). Change subject: KUDU-1643 Prune hash partitions based on IN-list predicates B ..

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

2016-11-24 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 13: (29 comments)

[kudu-CR] KUDU-1643 Prune hash partitions based on IN-list predicates B

2016-11-24 Thread Haijie Hong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5176 to look at the new patch set (#5). Change subject: KUDU-1643 Prune hash partitions based on IN-list predicates B ..

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

2016-11-24 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 (#13). Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files ..

[kudu-CR] KUDU-1643 Prune hash partitions based on IN-list predicates B

2016-11-24 Thread Haijie Hong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5176 to look at the new patch set (#4). Change subject: KUDU-1643 Prune hash partitions based on IN-list predicates B ..

[kudu-CR] KUDU-1643 Prune hash partitions based on IN-list predicates B

2016-11-24 Thread Haijie Hong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5176 to look at the new patch set (#3). Change subject: KUDU-1643 Prune hash partitions based on IN-list predicates B ..