[kudu-CR] Workaround test failures running with MIT krb5 1.10

Adar Dembo has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 8: Code-Review+2 (1 comment)

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Todd Lipcon has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/4990/7/CMakeLists.txt File CMakeLists.txt:

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Alexey Serbin has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/4990/7/CMakeLists.txt File CMakeLists.txt:

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Hello Dan Burkert, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4990 to look at the new patch set (#8). Change subject: Workaround test failures running with MIT krb5 1.10

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Todd Lipcon has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 7: (8 comments) http://gerrit.cloudera.org:8080/#/c/4990/7/CMakeLists.txt File CMakeLists.txt:

[kudu-CR] KUDU-180. Fix handling of defaults when creating tables

Dan Burkert has posted comments on this change. Change subject: KUDU-180. Fix handling of defaults when creating tables .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5031/1/src/kudu/common/common.proto File

[kudu-CR] Check sanity of standard file descriptors when starting daemons

Todd Lipcon has posted comments on this change. Change subject: Check sanity of standard file descriptors when starting daemons .. Patch Set 1: I suppose a death test would be possible, but it's such a rare scenario I don't

[kudu-CR] Check sanity of standard file descriptors when starting daemons

Todd Lipcon has submitted this change and it was merged. Change subject: Check sanity of standard file descriptors when starting daemons .. Check sanity of standard file descriptors when starting daemons This adds a sanity

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

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 (#10). 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

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 (#9). 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

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

[kudu-CR] [java] Add checkstyle coverage to Java build

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5016 to look at the new patch set (#3). Change subject: [java] Add checkstyle coverage to Java build .. [java] Add

[kudu-CR] [java] Add checkstyle coverage to Java build

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5016 to look at the new patch set (#2). Change subject: [java] Add checkstyle coverage to Java build .. [java] Add

[kudu-CR] env: remove EnvWrapper

Adar Dembo has posted comments on this change. Change subject: env: remove EnvWrapper .. Patch Set 1: Verified+1 Overriding Jenkins, ran into another instance of KUDU-1624. -- To view, visit

[kudu-CR] Check sanity of standard file descriptors when starting daemons

Jean-Daniel Cryans has posted comments on this change. Change subject: Check sanity of standard file descriptors when starting daemons .. Patch Set 1: Code-Review+1 All java tests pass on macOS with this patch, as they should.

[kudu-CR] Check sanity of standard file descriptors when starting daemons

Adar Dembo has posted comments on this change. Change subject: Check sanity of standard file descriptors when starting daemons .. Patch Set 1: Code-Review+2 (1 comment) Ah, I was hoping this could be tested with a death test.

[kudu-CR] Check sanity of standard file descriptors when starting daemons

Hello Jean-Daniel Cryans, Adar Dembo, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5030 to review the following change. Change subject: Check sanity of standard file descriptors when starting daemons

[kudu-CR] env: remove EnvWrapper

Hello Mike Percy, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5027 to review the following change. Change subject: env: remove EnvWrapper .. env: remove EnvWrapper We're

[kudu-CR] test util: prefer test dir over GetTestDataDirectory()

Hello Mike Percy, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5028 to review the following change. Change subject: test_util: prefer test_dir_ over GetTestDataDirectory() ..

[kudu-CR] log block manager: push down container locks

Hello Mike Percy, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5029 to review the following change. Change subject: log block manager: push down container locks .. log block

[kudu-CR] C++ client: hide deflate symbols

Adar Dembo has submitted this change and it was merged. Change subject: C++ client: hide deflate symbols .. C++ client: hide deflate symbols These snuck in via commit 8a4bb33. Oddly, client_symbol-test.sh was only observed to

[kudu-CR] log block manager: fix when append exceeds preallocated space

Adar Dembo has submitted this change and it was merged. Change subject: log block manager: fix when append exceeds preallocated space .. log block manager: fix when append exceeds preallocated space In commit abea8c6 I added a

[kudu-CR] [docs] Python updates for development page

Jordan Birdsell has posted comments on this change. Change subject: [docs] Python updates for development page .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/4586/1/docs/developing.adoc File

[kudu-CR] [docs] Python updates for development page

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4586 to look at the new patch set (#3). Change subject: [docs] Python updates for development page .. [docs] Python

[kudu-CR] log block manager: fix when append exceeds preallocated space

Jean-Daniel Cryans has posted comments on this change. Change subject: log block manager: fix when append exceeds preallocated space .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5024 To

[kudu-CR] Consolidate Row/CompactionInputRow printing on compaction

David Ribeiro Alves has posted comments on this change. Change subject: Consolidate Row/CompactionInputRow printing on compaction .. Patch Set 3: (1 comment)

[kudu-CR] Consolidate Row/CompactionInputRow printing on compaction

Hello Dinesh Bhat, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4988 to look at the new patch set (#6). Change subject: Consolidate Row/CompactionInputRow printing on compaction

[kudu-CR] log block manager: fix when append exceeds preallocated space

Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5024 to review the following change. Change subject: log block manager: fix when append exceeds preallocated space

[kudu-CR] C++ client: hide deflate symbols

Todd Lipcon has posted comments on this change. Change subject: C++ client: hide deflate symbols .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5023 To unsubscribe, visit

[kudu-CR] C++ client: hide deflate symbols

Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5023 to review the following change. Change subject: C++ client: hide deflate symbols .. C++ client: hide deflate symbols

[kudu-CR] Reject CREATE TABLE ops with even replication factor

Alexey Serbin has posted comments on this change. Change subject: Reject CREATE TABLE ops with even replication factor .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4945 To unsubscribe, visit

[kudu-CR] Add a macro to LOG and return on a non-OK status

David Ribeiro Alves has submitted this change and it was merged. Change subject: Add a macro to LOG and return on a non-OK status .. Add a macro to LOG and return on a non-OK status We often see the following pattern: Status

[kudu-CR] Add a macro to LOG and return on a non-OK status

Alexey Serbin has posted comments on this change. Change subject: Add a macro to LOG and return on a non-OK status .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4927 To unsubscribe, visit

[kudu-CR] Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param

David Ribeiro Alves has posted comments on this change. Change subject: Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param .. Patch Set 8: Code-Review+2 just a rebase,

[kudu-CR] Improve debuggability of the delta/compaction path

David Ribeiro Alves has posted comments on this change. Change subject: Improve debuggability of the delta/compaction path .. Patch Set 6: Code-Review+2 just a rebase, keeping +2 -- To view, visit

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

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

[kudu-CR] Consolidate Row/CompactionInputRow printing on compaction

Hello Dinesh Bhat, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4988 to look at the new patch set (#5). Change subject: Consolidate Row/CompactionInputRow printing on compaction

[kudu-CR] Consolidate Row/CompactionInputRow printing on compaction

David Ribeiro Alves has posted comments on this change. Change subject: Consolidate Row/CompactionInputRow printing on compaction .. Patch Set 3: (4 comments)

[kudu-CR] Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param

Jean-Daniel Cryans has posted comments on this change. Change subject: Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] Improve debuggability of the delta/compaction path

Jean-Daniel Cryans has posted comments on this change. Change subject: Improve debuggability of the delta/compaction path .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4930 To unsubscribe,

[kudu-CR] [java] Add checkstyle coverage to Java build

Jean-Daniel Cryans has posted comments on this change. Change subject: [java] Add checkstyle coverage to Java build .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5016/1/java/kudu_style.xml File

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Adar Dembo has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 7: (7 comments) http://gerrit.cloudera.org:8080/#/c/4990/7/CMakeLists.txt File CMakeLists.txt:

[kudu-CR] Improve debuggability of the delta/compaction path

David Ribeiro Alves has posted comments on this change. Change subject: Improve debuggability of the delta/compaction path .. Patch Set 5: Code-Review+2 Just a rebase, keeping alexey's +2 -- To view, visit

[kudu-CR] Consolidate Row/CompactionInputRow printing on compaction

Hello Dinesh Bhat, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4988 to look at the new patch set (#4). Change subject: Consolidate Row/CompactionInputRow printing on compaction

[kudu-CR] Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param

David Ribeiro Alves has posted comments on this change. Change subject: Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param .. Patch Set 5: (1 comment)

[kudu-CR] Rename LogicalClock::NowForMetrics() to GetCurrentTime()

Hello Dinesh Bhat, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4989 to look at the new patch set (#4). Change subject: Rename LogicalClock::NowForMetrics() to GetCurrentTime()

[kudu-CR] Improve debuggability of the delta/compaction path

Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4930 to look at the new patch set (#5). Change subject: Improve debuggability of the delta/compaction path

[kudu-CR] KUDU-237 (part 1) - Support proper mutation encoding for reinserts

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4791 to look at the new patch set (#13). Change subject: KUDU-237 (part 1) - Support proper mutation encoding for reinserts

[kudu-CR] Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param

David Ribeiro Alves has uploaded a new patch set (#7). Change subject: Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param .. Make

[kudu-CR] Add a macro to LOG and return on a non-OK status

David Ribeiro Alves has posted comments on this change. Change subject: Add a macro to LOG and return on a non-OK status .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/4927/5//COMMIT_MSG Commit Message:

[kudu-CR] Add a macro to LOG and return on a non-OK status

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4927 to look at the new patch set (#6). Change subject: Add a macro to LOG and return on a non-OK status .. Add a macro

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

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

[kudu-CR] KUDU-237 (part 1) - Support proper mutation encoding for reinserts

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4791 to look at the new patch set (#12). Change subject: KUDU-237 (part 1) - Support proper mutation encoding for reinserts

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Alexey Serbin has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/4990/5/src/kudu/security/init.cc File

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Alexey Serbin has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 7: Code-Review+1 (3 comments) http://gerrit.cloudera.org:8080/#/c/4990/7/CMakeLists.txt File

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Todd Lipcon has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 5: (7 comments) http://gerrit.cloudera.org:8080/#/c/4990/5/src/kudu/security/CMakeLists.txt File

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4990 to look at the new patch set (#7). Change subject: Workaround test failures running with MIT krb5 1.10

[kudu-CR] python: allow passing single strings instead of lists in more places

Jordan Birdsell has submitted this change and it was merged. Change subject: python: allow passing single strings instead of lists in more places .. python: allow passing single strings instead of lists in more places Several

[kudu-CR] Allow binaries built on el6.6+ to run on el6.4

Adar Dembo has posted comments on this change. Change subject: Allow binaries built on el6.6+ to run on el6.4 .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5011 To unsubscribe, visit

[kudu-CR] Allow binaries built on el6.6+ to run on el6.4

Adar Dembo has submitted this change and it was merged. Change subject: Allow binaries built on el6.6+ to run on el6.4 .. Allow binaries built on el6.6+ to run on el6.4 This provides a workaround to an OpenSSL ABI

[kudu-CR] [java] Add checkstyle coverage to Java build

Will Berkeley has posted comments on this change. Change subject: [java] Add checkstyle coverage to Java build .. Patch Set 1: (1 comment)

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4990 to look at the new patch set (#6). Change subject: Workaround test failures running with MIT krb5 1.10

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Todd Lipcon has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/4990/5/src/kudu/security/init.cc File

[kudu-CR] [doc] mention the loadgen tool in the release notes

Adar Dembo has posted comments on this change. Change subject: [doc] mention the loadgen tool in the release notes .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5020 To unsubscribe, visit

[kudu-CR] [doc] mention the loadgen tool in the release notes

Adar Dembo has submitted this change and it was merged. Change subject: [doc] mention the loadgen tool in the release notes .. [doc] mention the loadgen tool in the release notes Added information about the load generation

[kudu-CR] [java] Add checkstyle coverage to Java build

Dan Burkert has posted comments on this change. Change subject: [java] Add checkstyle coverage to Java build .. Patch Set 1: (1 comment) I'm a big +1 on this, just had the one nit.

[kudu-CR] python: allow passing single strings instead of lists in more places

Jordan Birdsell has posted comments on this change. Change subject: python: allow passing single strings instead of lists in more places .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5019 To

[kudu-CR] Add a macro to LOG and return on a non-OK status

Alexey Serbin has posted comments on this change. Change subject: Add a macro to LOG and return on a non-OK status .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/4927/5//COMMIT_MSG Commit Message: PS5, Line

[kudu-CR] python: check status result when setting row contents

Jordan Birdsell has posted comments on this change. Change subject: python: check status result when setting row contents .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5018/1/python/kudu/client.pyx File

[kudu-CR] [doc] mention the loadgen tool in the release notes

Hello Dinesh Bhat, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5020 to look at the new patch set (#2). Change subject: [doc] mention the loadgen tool in the release notes .. [doc]

[kudu-CR] Release notes for the Java client in 1.1

Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Release notes for the Java client in 1.1 .. Release notes for the Java client in 1.1 Change-Id: Idf04d1db4769e19e6421628bbe980cb4c058c62c

[kudu-CR] [doc] mention the loadgen tool in the release notes

Dinesh Bhat has posted comments on this change. Change subject: [doc] mention the loadgen tool in the release notes .. Patch Set 1: Code-Review+1 (3 comments) http://gerrit.cloudera.org:8080/#/c/5020/1/docs/release_notes.adoc

[kudu-CR] [doc] mention the loadgen tool in the release notes

Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5020 Change subject: [doc] mention the loadgen tool in the release notes .. [doc] mention the loadgen tool in the release notes Added

[kudu-CR] Allow binaries built on el6.6+ to run on el6.4

Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5011 to look at the new patch set (#2). Change subject: Allow binaries built on el6.6+ to run on el6.4 ..

[kudu-CR] python: allow passing single strings instead of lists in more places

Hello Jordan Birdsell, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5019 to review the following change. Change subject: python: allow passing single strings instead of lists in more places

[kudu-CR] python: check status result when setting row contents

Hello Jordan Birdsell, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5018 to review the following change. Change subject: python: check status result when setting row contents .. python:

[kudu-CR] Release notes for the Java client in 1.1

Jean-Daniel Cryans has posted comments on this change. Change subject: Release notes for the Java client in 1.1 .. Patch Set 4: Had to rebase and solve conflicts due to the other release notes that came in. David, you good

[kudu-CR] Release notes for the Java client in 1.1

Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5002 to look at the new patch set (#4). Change subject: Release notes for the Java client in 1.1 .. Release

[kudu-CR] Consolidate Row/CompactionInputRow printing on compaction

Alexey Serbin has posted comments on this change. Change subject: Consolidate Row/CompactionInputRow printing on compaction .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/4988/3/src/kudu/tablet/compaction.cc

[kudu-CR] Workaround test failures running with MIT krb5 1.10

Todd Lipcon has posted comments on this change. Change subject: Workaround test failures running with MIT krb5 1.10 .. Patch Set 5: > Why? I would expect that credential passing between the embedder and the > client will be

[kudu-CR](gh-pages) [site] - Update committer page

Jordan Birdsell has uploaded a new patch set (#2). Change subject: [site] - Update committer page .. [site] - Update committer page Updating committer page with Jordan's name. Change-Id:

[kudu-CR](gh-pages) [site] - Update committer page

Will Berkeley has posted comments on this change. Change subject: [site] - Update committer page .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5017 To unsubscribe, visit

[kudu-CR](gh-pages) [site] - Update committer page

Will Berkeley has submitted this change and it was merged. Change subject: [site] - Update committer page .. [site] - Update committer page Updating committer page with Jordan's name. Change-Id:

[kudu-CR](gh-pages) [site] - Update committer page

Will Berkeley has posted comments on this change. Change subject: [site] - Update committer page .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5017 To unsubscribe, visit

[kudu-CR](gh-pages) [site] - Update committer page

Will Berkeley has posted comments on this change. Change subject: [site] - Update committer page .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5017/1/committers.md File committers.md: PS1, Line 22: jfarrell

[kudu-CR](gh-pages) [site] - Update committer page

Jordan Birdsell has uploaded a new change for review. http://gerrit.cloudera.org:8080/5017 Change subject: [site] - Update committer page .. [site] - Update committer page Updating committer page with Jordan's name.