[kudu-CR] Kudu Jepsen Tests - Initial Commit

Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (1 comment) BTW, what would be the best way to update this patch given the fact there are subsequent ones?

[kudu-CR] Kudu Jepsen Tests - Initial Commit

Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (4 comments) http://gerrit.cloudera.org:8080/#/c/5492/12/build-support/jenkins/build-and-test.sh File

[kudu-CR] Add Google Breakpad support to Kudu

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5620 to look at the new patch set (#3). Change subject: Add Google Breakpad support to Kudu .. Add Google Breakpad

[kudu-CR] Add Google Breakpad support to Kudu

Mike Percy has posted comments on this change. Change subject: Add Google Breakpad support to Kudu .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5620/1//COMMIT_MSG Commit Message: Line 28: because

[kudu-CR] [jepsen.kudu] run tests from the clojure-maven-plugin

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5500 to look at the new patch set (#10). Change subject: [jepsen.kudu] run tests from the clojure-maven-plugin ..

[kudu-CR] [kudu-jepsen] added Jenkins script

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5624 to look at the new patch set (#3). Change subject: [kudu-jepsen] added Jenkins script .. [kudu-jepsen] added

[kudu-CR] EMC: Don't reuse data dir for log dir

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5619 to look at the new patch set (#3). Change subject: EMC: Don't reuse data dir for log dir .. EMC: Don't reuse data

[kudu-CR] lsan: Add macro to ignore a leaked object

Mike Percy has submitted this change and it was merged. Change subject: lsan: Add macro to ignore a leaked object .. lsan: Add macro to ignore a leaked object Change-Id: I370c035e2b219de6676b99d564102d77fcaef570 Reviewed-on:

[kudu-CR] env util: Implement CreateDirsRecursively()

Mike Percy has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/5618/2/src/kudu/util/env_util-test.cc File

[kudu-CR] env: Add GetCurrentWorkingDir() and ChangeDir()

Mike Percy has abandoned this change. Change subject: env: Add GetCurrentWorkingDir() and ChangeDir() .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/5629 To unsubscribe, visit

[kudu-CR] env: Add GetCurrentWorkingDir() and ChangeDir()

Mike Percy has posted comments on this change. Change subject: env: Add GetCurrentWorkingDir() and ChangeDir() .. Patch Set 1: (1 comment) Merging into https://gerrit.cloudera.org/5618

[kudu-CR] path util: Add SplitPath() helper function

Mike Percy has abandoned this change. Change subject: path_util: Add SplitPath() helper function .. Abandoned Merging into https://gerrit.cloudera.org/5618 -- To view, visit http://gerrit.cloudera.org:8080/5628 To

[kudu-CR] path util: Add SplitPath() helper function

Mike Percy has posted comments on this change. Change subject: path_util: Add SplitPath() helper function .. Patch Set 1: (2 comments) I am going to merge this into the CreateDirsRecursively() CR per Todd's request.

[kudu-CR] env util: Implement CreateDirsRecursively()

Adar Dembo has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5618/2/src/kudu/util/env_util.cc File

[kudu-CR] env util: Implement CreateDirsRecursively()

Todd Lipcon has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 2: (2 comments) general note: while I like fine-grained patches, for these simple utility functions I

[kudu-CR] path util: Add IsRelativePathOrEmpty() helper function

Todd Lipcon has posted comments on this change. Change subject: path_util: Add IsRelativePathOrEmpty() helper function .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5617/1/src/kudu/util/path_util.cc File

[kudu-CR] env: Add GetCurrentWorkingDir() and ChangeDir()

Adar Dembo has posted comments on this change. Change subject: env: Add GetCurrentWorkingDir() and ChangeDir() .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5629/1/src/kudu/util/env_posix.cc File

[kudu-CR] Kudu Jepsen Tests - Initial Commit

Todd Lipcon has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (18 comments) http://gerrit.cloudera.org:8080/#/c/5492/12/build-support/jenkins/build-and-test.sh File

[kudu-CR] path util: Add SplitPath() helper function

Adar Dembo has posted comments on this change. Change subject: path_util: Add SplitPath() helper function .. Patch Set 1: (2 comments) Still not sure whether we should allow "empty" paths, but the approach you took here

[kudu-CR] Add Google Breakpad support to Kudu

Mike Percy has posted comments on this change. Change subject: Add Google Breakpad support to Kudu .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/5620/1//COMMIT_MSG Commit Message: Line 28: because

[kudu-CR] lsan: Add macro to ignore a leaked object

Adar Dembo has posted comments on this change. Change subject: lsan: Add macro to ignore a leaked object .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5616 To unsubscribe, visit

[kudu-CR] path util: Add IsRelativePathOrEmpty() helper function

Adar Dembo has posted comments on this change. Change subject: path_util: Add IsRelativePathOrEmpty() helper function .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5617/1/src/kudu/util/path_util.cc File

[kudu-CR] EMC: Don't reuse data dir for log dir

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5619 to look at the new patch set (#2). Change subject: EMC: Don't reuse data dir for log dir .. EMC: Don't reuse data

[kudu-CR] lsan: Add macro to ignore a leaked object

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5616 to look at the new patch set (#3). Change subject: lsan: Add macro to ignore a leaked object .. lsan: Add macro to

[kudu-CR] env: Add GetCurrentWorkingDir() and ChangeDir()

Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/5629 Change subject: env: Add GetCurrentWorkingDir() and ChangeDir() .. env: Add GetCurrentWorkingDir() and ChangeDir() These are functions

[kudu-CR] env util: Implement CreateDirsRecursively()

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5618 to look at the new patch set (#2). Change subject: env_util: Implement CreateDirsRecursively() .. env_util:

[kudu-CR](branch-1.2.x) [doxygen] omit docs for forward declarations

Todd Lipcon has posted comments on this change. Change subject: [doxygen] omit docs for forward declarations .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5622 To unsubscribe, visit

[kudu-CR] lsan: Add macro to ignore a leaked object

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5616 to look at the new patch set (#2). Change subject: lsan: Add macro to ignore a leaked object .. lsan: Add macro to

[kudu-CR] path util: Add IsRelativePathOrEmpty() helper function

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5617 to look at the new patch set (#2). Change subject: path_util: Add IsRelativePathOrEmpty() helper function ..

[kudu-CR] env util: Implement CreateDirsRecursively()

Mike Percy has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5618/1/src/kudu/util/env_util.cc File

[kudu-CR] Add Google Breakpad support to Kudu

Todd Lipcon has posted comments on this change. Change subject: Add Google Breakpad support to Kudu .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/5620/1//COMMIT_MSG Commit Message: Line 28: because

[kudu-CR] [jenkins] added JEPSEN build configuration

Alexey Serbin has abandoned this change. Change subject: [jenkins] added JEPSEN build configuration .. Abandoned Abandoned in favor of http://gerrit.cloudera.org:8080/5624 -- To view, visit

[kudu-CR] update

Alexey Serbin has abandoned this change. Change subject: update .. Abandoned Please ignore -- wrong patch -- To view, visit http://gerrit.cloudera.org:8080/5623 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[kudu-CR] [jepsen.kudu] run tests from the clojure-maven-plugin

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5500 to look at the new patch set (#9). Change subject: [jepsen.kudu] run tests from the clojure-maven-plugin ..

[kudu-CR] [kudu-jepsen] added Jenkins script

Alexey Serbin has uploaded a new patch set (#2). Change subject: [kudu-jepsen] added Jenkins script .. [kudu-jepsen] added Jenkins script Added a shell script to build Kudu and run kudu-jepsen tests. Currently, the set of

[kudu-CR] EMC: Don't reuse data dir for log dir

Mike Percy has posted comments on this change. Change subject: EMC: Don't reuse data dir for log dir .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5619/1/src/kudu/integration-tests/external_mini_cluster.cc

[kudu-CR] update

Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5623 Change subject: update .. update Change-Id: I25d5c94e432df42ea274ce9a94a692996ab9a498 --- M java/kudu-jepsen/pom.xml 1 file changed, 6

[kudu-CR] [kudu-jepsen] added Jenkins script

Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5624 Change subject: [kudu-jepsen] added Jenkins script .. [kudu-jepsen] added Jenkins script Added a shell script to build Kudu and run

[kudu-CR] env util: Implement CreateDirsRecursively()

Mike Percy has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/5618/1/src/kudu/util/env_util-test.cc File

[kudu-CR] EMC: Don't reuse data dir for log dir

Todd Lipcon has posted comments on this change. Change subject: EMC: Don't reuse data dir for log dir .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5619/1//COMMIT_MSG Commit Message: Line 7: EMC: Don't

[kudu-CR] path util: Add IsRelativePath() helper function

Mike Percy has posted comments on this change. Change subject: path_util: Add IsRelativePath() helper function .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5617/1/src/kudu/util/path_util.cc File

[kudu-CR] lsan: Add macro to ignore a leaked object

Mike Percy has posted comments on this change. Change subject: lsan: Add macro to ignore a leaked object .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5616/1/src/kudu/util/debug/leakcheck_disabler.h File

[kudu-CR] env util: Implement CreateDirsRecursively()

Adar Dembo has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5618/1/src/kudu/util/env_util.cc File

[kudu-CR] env util: Implement CreateDirsRecursively()

Todd Lipcon has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/5618/1/src/kudu/util/env_util-test.cc File

[kudu-CR] path util: Add IsRelativePath() helper function

Adar Dembo has posted comments on this change. Change subject: path_util: Add IsRelativePath() helper function .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5617/1/src/kudu/util/path_util.cc File

[kudu-CR] path util: Add IsRelativePath() helper function

Todd Lipcon has posted comments on this change. Change subject: path_util: Add IsRelativePath() helper function .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5617/1/src/kudu/util/path_util.cc File

[kudu-CR] lsan: Add macro to ignore a leaked object

Todd Lipcon has posted comments on this change. Change subject: lsan: Add macro to ignore a leaked object .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5616/1/src/kudu/util/debug/leakcheck_disabler.h File

[kudu-CR] KUDU-734. Add test coverage for encodings and strings

Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-734. Add test coverage for encodings and strings .. KUDU-734. Add test coverage for encodings and strings Added a testAllPrimaryKeyTypes test to

[kudu-CR] KUDU-734. Add test coverage for encodings and strings

Todd Lipcon has posted comments on this change. Change subject: KUDU-734. Add test coverage for encodings and strings .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5588 To unsubscribe, visit

[kudu-CR] path util: Add IsRelativePath() helper function

Hello Dinesh Bhat, Lars Volker, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5617 to review the following change. Change subject: path_util: Add IsRelativePath() helper function

[kudu-CR] env util: Implement CreateDirsRecursively()

Hello Dinesh Bhat, Lars Volker, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5618 to review the following change. Change subject: env_util: Implement CreateDirsRecursively()

[kudu-CR] Add Google Breakpad support to Kudu

Hello Dinesh Bhat, Lars Volker, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5620 to review the following change. Change subject: Add Google Breakpad support to Kudu .. Add

[kudu-CR] [doxygen] omit docs for forward declarations

Todd Lipcon has submitted this change and it was merged. Change subject: [doxygen] omit docs for forward declarations .. [doxygen] omit docs for forward declarations Change-Id: I6a560f6a92202f0e7c1329fc3952c12c4fabba9f

[kudu-CR] lsan: Add macro to ignore a leaked object

Hello Dinesh Bhat, Lars Volker, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5616 to review the following change. Change subject: lsan: Add macro to ignore a leaked object ..

[kudu-CR] [doxygen] omit docs for forward declarations

Alexey Serbin has posted comments on this change. Change subject: [doxygen] omit docs for forward declarations .. Patch Set 1: > Assuming this is sufficient (i.e. you've looked at the generated > docs), looks good. Thank you

[kudu-CR] [doxygen] omit docs for forward declarations

Adar Dembo has posted comments on this change. Change subject: [doxygen] omit docs for forward declarations .. Patch Set 1: Code-Review+2 Assuming this is sufficient (i.e. you've looked at the generated docs), looks good. --

[kudu-CR] [doxygen] omit docs for forward declarations

Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5614 Change subject: [doxygen] omit docs for forward declarations .. [doxygen] omit docs for forward declarations Change-Id:

[kudu-CR](branch-1.2.x) KUDU-1817. Fix kudu-spark2 artifactId in pom.xml

Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1817. Fix kudu-spark2 artifactId in pom.xml .. KUDU-1817. Fix kudu-spark2 artifactId in pom.xml kudu-spark2 pom.xml is malformed due to a

[kudu-CR](branch-1.2.x) KUDU-1817. Fix kudu-spark2 artifactId in pom.xml

Todd Lipcon has posted comments on this change. Change subject: KUDU-1817. Fix kudu-spark2 artifactId in pom.xml .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5612 To unsubscribe, visit

[kudu-CR](branch-1.2.x) KUDU-1817. Fix kudu-spark2 artifactId in pom.xml

Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/5612 Change subject: KUDU-1817. Fix kudu-spark2 artifactId in pom.xml .. KUDU-1817. Fix kudu-spark2 artifactId in pom.xml kudu-spark2

[kudu-CR] thirdparty: Link curl against el6 workaround openssl

Adar Dembo has submitted this change and it was merged. Change subject: thirdparty: Link curl against el6 workaround openssl .. thirdparty: Link curl against el6 workaround openssl This patch enables HTTPS support for

[kudu-CR] thirdparty: Link curl against el6 workaround openssl

Adar Dembo has posted comments on this change. Change subject: thirdparty: Link curl against el6 workaround openssl .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5603 To unsubscribe, visit

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

Todd Lipcon has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: Chris and Dan -- now that the holidays are behind us, any further thoughts on this

[kudu-CR] KUDU-1733. Update consistency semantics doc

Todd Lipcon has posted comments on this change. Change subject: KUDU-1733. Update consistency semantics doc .. Patch Set 1: Woops, I forgot about that old patch. I'll try to incorporate it. -- To view, visit

[kudu-CR] Initial draft of release notes and doc updates for 1.2

Jean-Daniel Cryans has posted comments on this change. Change subject: Initial draft of release notes and doc updates for 1.2 .. Patch Set 2: Do you have a rendering? -- To view, visit http://gerrit.cloudera.org:8080/5604 To

[kudu-CR] KUDU-1733. Update consistency semantics doc

Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1733. Update consistency semantics doc .. Patch Set 1: I didn't do a diff but did you look at https://gerrit.cloudera.org/#/c/1504/ ? -- To view,

[kudu-CR] Initial draft of release notes and doc updates for 1.2

Jean-Daniel Cryans has posted comments on this change. Change subject: Initial draft of release notes and doc updates for 1.2 .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5604/2/docs/release_notes.adoc File

[kudu-CR] KUDU-1822. Update docs to build the documentation in ubuntu

Todd Lipcon has posted comments on this change. Change subject: KUDU-1822. Update docs to build the documentation in ubuntu .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5608 To unsubscribe,

[kudu-CR] KUDU-1822. Update docs to build the documentation in ubuntu

Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1822. Update docs to build the documentation in ubuntu .. KUDU-1822. Update docs to build the documentation in ubuntu Include two packages, gem and

[kudu-CR] KUDU-1822. Update docs to build the documentation in ubuntu

Jun He has uploaded a new change for review. http://gerrit.cloudera.org:8080/5608 Change subject: KUDU-1822. Update docs to build the documentation in ubuntu .. KUDU-1822. Update docs to build the documentation in ubuntu

[kudu-CR] KUDU-734. Add test coverage for encodings and strings

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5588 to look at the new patch set (#2). Change subject: KUDU-734. Add test coverage for encodings and strings ..