[kudu-CR] [docs] Add release note for CSV/Avro/Parquet import/export

2017-08-25 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new patch set (#2). Change subject: [docs] Add release note for CSV/Avro/Parquet import/export .. [docs] Add release note for CSV/Avro/Parquet import/export Release notes for: 5d53a3b7146c6bcf330ddae0

[kudu-CR] [docs] Add release note for CSV/Avro/Parquet import/export

2017-08-25 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7852 Change subject: [docs] Add release note for CSV/Avro/Parquet import/export .. [docs] Add release note for CSV/Avro/Parquet import/export

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] java-client: improve error messages when failing to connect to secure cluster

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: java-client: improve error messages when failing to connect to secure cluster .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7824/2//COMMIT_MSG

[kudu-CR] java-client: improve error messages when failing to connect to secure cluster

2017-08-25 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7824 to look at the new patch set (#3). Change subject: java-client: improve error messages when failing to connect to secure cluster

[kudu-CR] KUDU-1976. java: use ephemeral ports for MiniKDC

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1976. java: use ephemeral ports for MiniKDC .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1976. java: use ephemeral ports for MiniKDC

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1976. java: use ephemeral ports for MiniKDC .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2114. Don't re-delete tombstoned replicas

2017-08-25 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7842 to look at the new patch set (#2). Change subject: KUDU-2114. Don't re-delete tombstoned replicas .. KUDU-2114.

[kudu-CR] KUDU-1976. java: use ephemeral ports for MiniKDC

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1976. java: use ephemeral ports for MiniKDC .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1976. java: use ephemeral ports for MiniKDC

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1976. java: use ephemeral ports for MiniKDC .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1976. java: use ephemeral ports for MiniKDC

2017-08-25 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7850 to look at the new patch set (#2). Change subject: KUDU-1976. java: use ephemeral ports for MiniKDC .. KUDU-1976.

[kudu-CR] KUDU-1976. java: use ephemeral ports for MiniKDC

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1976. java: use ephemeral ports for MiniKDC .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1894 fixed deadlock in client.Connection

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1894 fixed deadlock in client.Connection .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1976. java: use ephemeral ports for MiniKDC

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1976. java: use ephemeral ports for MiniKDC .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1544: Race in Java client's AsyncKuduSession.apply()

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1544: Race in Java client's AsyncKuduSession.apply() .. KUDU-1544: Race in Java client's AsyncKuduSession.apply() This fixes a potential race in

[kudu-CR] KUDU-2114. Don't re-delete tombstoned replicas

2017-08-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-2114. Don't re-delete tombstoned replicas .. Patch Set 1: (8 comments)

[kudu-CR] release notes: add web UI config dashboard entry

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: release notes: add web UI config dashboard entry .. release notes: add web UI config dashboard entry Change-Id: I8812e1643afc50af0633343aa76fbea79bc83d72

[kudu-CR] release notes: add contributor counts

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: release notes: add contributor counts .. release notes: add contributor counts Change-Id: Ib385b93671056c37474367266291370d7893550c Reviewed-on:

[kudu-CR] KUDU-1976. java: use ephemeral ports for MiniKDC

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1976. java: use ephemeral ports for MiniKDC .. Patch Set 1: (1 comment)

[kudu-CR] [tests] fix flakiness in catalog manager tsk-itest

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [tests] fix flakiness in catalog_manager_tsk-itest .. [tests] fix flakiness in catalog_manager_tsk-itest After recent updates the

[kudu-CR] [tests] fix flakiness in catalog manager tsk-itest

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [tests] fix flakiness in catalog_manager_tsk-itest .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7823 To unsubscribe, visit

[kudu-CR](branch-1.4.x) [build-support] fixed IWYU no-op configuration

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [build-support] fixed IWYU no-op configuration .. [build-support] fixed IWYU no-op configuration Jenkins is upset if no output is produced by a build: no

[kudu-CR](branch-1.4.x) [build-support] fixed IWYU no-op configuration

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [build-support] fixed IWYU no-op configuration .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7831 To unsubscribe, visit

[kudu-CR] Disable the daemon usage in Jenkins Gradle build

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Disable the daemon usage in Jenkins Gradle build .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7841 To unsubscribe, visit

[kudu-CR](branch-1.2.x) [build-support] fixed IWYU no-op configuration

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [build-support] fixed IWYU no-op configuration .. [build-support] fixed IWYU no-op configuration Jenkins is upset if no output is produced by a build: no

[kudu-CR] Fix flakiness in ts recovery-itest TestCrashBeforeWriteLogSegmentHeader

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Fix flakiness in ts_recovery-itest TestCrashBeforeWriteLogSegmentHeader .. Fix flakiness in ts_recovery-itest TestCrashBeforeWriteLogSegmentHeader

[kudu-CR](branch-1.2.x) [build-support] fixed IWYU no-op configuration

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [build-support] fixed IWYU no-op configuration .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7833 To unsubscribe, visit

[kudu-CR] Disable the daemon usage in Jenkins Gradle build

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Disable the daemon usage in Jenkins Gradle build .. Disable the daemon usage in Jenkins Gradle build We enable the daemon by default for user builds but

[kudu-CR] Add tablet state summary metrics and fix KUDU-2044

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add tablet state summary metrics and fix KUDU-2044 .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7618/2//COMMIT_MSG Commit Message: PS2, Line

[kudu-CR] release notes: add web UI config dashboard entry

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: release notes: add web UI config dashboard entry .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7847 To unsubscribe, visit

[kudu-CR] release notes: add contributor counts

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: release notes: add contributor counts .. Patch Set 1: Code-Review+2 Verified+1 Nice touch -- To view, visit http://gerrit.cloudera.org:8080/7848 To unsubscribe,

[kudu-CR] KUDU-1976. java: use ephemeral ports for MiniKDC

2017-08-25 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/7850 to review the following change. Change subject: KUDU-1976. java: use ephemeral ports for MiniKDC ..

[kudu-CR] [rpc] faster generation of KRPC call ID

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: [rpc] faster generation of KRPC call ID .. Abandoned This is not needed: the current implementation seems to be good enough. -- To view, visit

[kudu-CR] KUDU-1544: Race in Java client's AsyncKuduSession.apply()

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1544: Race in Java client's AsyncKuduSession.apply() .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7839 To unsubscribe,

[kudu-CR] KUDU-2114. Don't re-delete tombstoned replicas

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2114. Don't re-delete tombstoned replicas .. Patch Set 1: (8 comments)

[kudu-CR] KUDU-1544: Race in Java client's AsyncKuduSession.apply()

2017-08-25 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7839 to look at the new patch set (#3). Change subject: KUDU-1544: Race in Java client's AsyncKuduSession.apply()

[kudu-CR] KUDU-1544: Race in Java client's AsyncKuduSession.apply()

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1544: Race in Java client's AsyncKuduSession.apply() .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently .. KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used

[kudu-CR] release notes: add web UI config dashboard entry

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new change for review. http://gerrit.cloudera.org:8080/7847 Change subject: release notes: add web UI config dashboard entry .. release notes: add web UI config dashboard entry Change-Id:

[kudu-CR] release notes: add contributor counts

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new change for review. http://gerrit.cloudera.org:8080/7848 Change subject: release notes: add contributor counts .. release notes: add contributor counts Change-Id:

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7840 To unsubscribe, visit

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. [release-notes] KUDU-2013 is fixed in Kudu 1.5 Change-Id: Ie3098b05747bc19b5d1a8f931d171cd4048c8bbf

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7840 To unsubscribe, visit

[kudu-CR] KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently .. Patch Set 2: Verified+1 Another false positive from IWYU, the rest of the build

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7840 To unsubscribe, visit

[kudu-CR] KUDU-2114. Don't re-delete tombstoned replicas

2017-08-25 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/7842 Change subject: KUDU-2114. Don't re-delete tombstoned replicas .. KUDU-2114. Don't re-delete tombstoned replicas A bug was introduced in

[kudu-CR] Fix flakiness in ts recovery-itest TestCrashBeforeWriteLogSegmentHeader

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Fix flakiness in ts_recovery-itest TestCrashBeforeWriteLogSegmentHeader .. Patch Set 2: > That's a safe bet :) > > I'm don't have a great explanation. I

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7840 To unsubscribe, visit

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7840/2/docs/release_notes.adoc File

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7840 to look at the new patch set (#3). Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 ..

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7840/2/docs/release_notes.adoc File

[kudu-CR] Disable the daemon usage in Jenkins Gradle build

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Disable the daemon usage in Jenkins Gradle build .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7841 To unsubscribe, visit

[kudu-CR] [docs] update on automatic authn token reacquisition

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [docs] update on automatic authn token reacquisition .. [docs] update on automatic authn token reacquisition Updated corresponding section in the docs

[kudu-CR](branch-1.3.x) [build-support] fixed IWYU no-op configuration

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [build-support] fixed IWYU no-op configuration .. [build-support] fixed IWYU no-op configuration Jenkins is upset if no output is produced by a build:

[kudu-CR] Disable the daemon usage in Jenkins Gradle build

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Disable the daemon usage in Jenkins Gradle build .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7841 To unsubscribe, visit

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#2). Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. [release-notes] KUDU-2013 is fixed in Kudu 1.5 Change-Id: Ie3098b05747bc19b5d1a8f931d171cd4048c8bbf --- M

[kudu-CR] [docs] update on automatic authn token reacquisition

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [docs] update on automatic authn token reacquisition .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7837 To unsubscribe, visit

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7840/1/docs/release_notes.adoc File

[kudu-CR] Disable the daemon usage in Jenkins Gradle build

2017-08-25 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new patch set (#2). Change subject: Disable the daemon usage in Jenkins Gradle build .. Disable the daemon usage in Jenkins Gradle build We enable the daemon by default for user builds but should

[kudu-CR] Fix flakiness in ts recovery-itest TestCrashBeforeWriteLogSegmentHeader

2017-08-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Fix flakiness in ts_recovery-itest TestCrashBeforeWriteLogSegmentHeader .. Patch Set 2: That's a safe bet :) I'm don't have a great explanation. I didn't do a

[kudu-CR] Fix flakiness in ts recovery-itest TestCrashBeforeWriteLogSegmentHeader

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Fix flakiness in ts_recovery-itest TestCrashBeforeWriteLogSegmentHeader .. Patch Set 2: Code-Review+2 BTW, do you have any reasonable explanation why this test

[kudu-CR](branch-1.3.x) [build-support] fixed IWYU no-op configuration

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [build-support] fixed IWYU no-op configuration .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7832 To unsubscribe, visit

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7840/1/docs/release_notes.adoc File

[kudu-CR] Fix flakiness in ts recovery-itest TestCrashBeforeWriteLogSegmentHeader

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix flakiness in ts_recovery-itest TestCrashBeforeWriteLogSegmentHeader .. Patch Set 2: Code-Review+2 I bet this patch had a high time:characters ratio! -- To

[kudu-CR] Disable the daemon usage in Jenkins Gradle build

2017-08-25 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7841 Change subject: Disable the daemon usage in Jenkins Gradle build .. Disable the daemon usage in Jenkins Gradle build We enable the daemon

[kudu-CR] [release-notes] KUDU-2013 is fixed in Kudu 1.5

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7840 Change subject: [release-notes] KUDU-2013 is fixed in Kudu 1.5 .. [release-notes] KUDU-2013 is fixed in Kudu 1.5 Change-Id:

[kudu-CR] KUDU-1544: Race in Java client's AsyncKuduSession.apply()

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1544: Race in Java client's AsyncKuduSession.apply() .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-1544: Race in Java client's AsyncKuduSession.apply()

2017-08-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1544: Race in Java client's AsyncKuduSession.apply() .. Patch Set 1: Code-Review+2 Did you loop TestAsyncKuduSession? -- To view, visit

[kudu-CR] [docs] update on automatic authn token reacquisition

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [docs] update on automatic authn token reacquisition .. Patch Set 1: Verified+1 Unrelated flake in TestAuthnTokenReacquireOpen: java.io.IOException: process

[kudu-CR] KUDU-1544: Race in Java client's AsyncKuduSession.apply()

2017-08-25 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/7839 to review the following change. Change subject: KUDU-1544: Race in Java client's AsyncKuduSession.apply()

[kudu-CR] java-client: improve error messages when failing to connect to secure cluster

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: java-client: improve error messages when failing to connect to secure cluster .. Patch Set 2: (3 comments) Looks good but the new test failed when running by

[kudu-CR] [docs] update on automatic authn token reacquisition

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [docs] update on automatic authn token reacquisition .. Patch Set 1: That's correct, updating release_notes.adoc is all that's necessary. -- To view, visit

[kudu-CR] Fix flakiness in ts recovery-itest TestCrashBeforeWriteLogSegmentHeader

2017-08-25 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new patch set (#2). Change subject: Fix flakiness in ts_recovery-itest TestCrashBeforeWriteLogSegmentHeader .. Fix flakiness in ts_recovery-itest TestCrashBeforeWriteLogSegmentHeader

[kudu-CR] KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: catalog_manager: don't log deleted tables/tablets at startup .. Patch Set 4: Verified+1 Overriding Jenkins, another Gradle daemon failure. -- To view, visit

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: catalog_manager: don't log deleted tables/tablets at startup .. catalog_manager: don't log deleted tables/tablets at startup On long-lived clusters this

[kudu-CR] Fix flakiness in ts recovery-itest TestCrashBeforeWriteLogSegmentHeader

2017-08-25 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new change for review. http://gerrit.cloudera.org:8080/7838 Change subject: Fix flakiness in ts_recovery-itest TestCrashBeforeWriteLogSegmentHeader .. Fix flakiness in ts_recovery-itest

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: catalog_manager: don't log deleted tables/tablets at startup .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7826 To unsubscribe,

[kudu-CR] KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently

2017-08-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently .. Patch Set 1: (5 comments) It looks like IWYU is giving false alarm. Maybe we can

[kudu-CR] KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently

2017-08-25 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7835 to look at the new patch set (#2). Change subject: KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: catalog_manager: don't log deleted tables/tablets at startup .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7826 To

[kudu-CR] Add metrics for tablet state and tablet copy

2017-08-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add metrics for tablet state and tablet copy .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7540 To unsubscribe, visit

[kudu-CR] [docs] update on automatic authn token reacquisition

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [docs] update on automatic authn token reacquisition .. Patch Set 1: Thanks, could you also add a release note that this has been fixed? -- To view, visit

[kudu-CR] KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently .. Patch Set 1: (5 comments)

[kudu-CR] [docs] update on automatic authn token reacquisition

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/7837 Change subject: [docs] update on automatic authn token reacquisition .. [docs] update on automatic authn token reacquisition Updated

[kudu-CR] java-client: improve error messages when failing to connect to secure cluster

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: java-client: improve error messages when failing to connect to secure cluster .. Patch Set 1: (5 comments) good point - test included.

[kudu-CR] java-client: improve error messages when failing to connect to secure cluster

2017-08-25 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7824 to look at the new patch set (#2). Change subject: java-client: improve error messages when failing to connect to secure cluster

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Alexey Serbin, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7826 to look at the new patch set (#4). Change subject: catalog_manager: don't log deleted tables/tablets at startup

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: catalog_manager: don't log deleted tables/tablets at startup .. Patch Set 3: Verified+1 Overriding Jenkins, another Gradle daemon failure. -- To view, visit

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: catalog_manager: don't log deleted tables/tablets at startup .. Patch Set 2: (1 comment)

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: catalog_manager: don't log deleted tables/tablets at startup .. Patch Set 3: Code-Review+2 (1 comment)

[kudu-CR] KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently

2017-08-25 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/7835 Change subject: KUDU-2102: fix PosixRWFile::Sync to guarantee durability when used concurrently .. KUDU-2102: fix PosixRWFile::Sync to

[kudu-CR] feat: add the wrapper for the Iterator by using InterruptibleIterator

2017-08-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: feat: add the wrapper for the Iterator by using InterruptibleIterator .. Patch Set 2: That's a good point about killTaskIfInterrupted being private to spark. I

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7826 to look at the new patch set (#3). Change subject: catalog_manager: don't log deleted tables/tablets at startup ..

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: catalog_manager: don't log deleted tables/tablets at startup .. Patch Set 2: (8 comments) > How confident are we that the Substitutes won't be more eagerly >

[kudu-CR] java-client: improve error messages when failing to connect to secure cluster

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: java-client: improve error messages when failing to connect to secure cluster .. Patch Set 1: > (5 comments) > > looks good, just a few nits Ah, and

[kudu-CR](branch-1.3.x) [build-support] fixed IWYU no-op configuration

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [build-support] fixed IWYU no-op configuration .. Patch Set 2: Verified+1 unrelated flake in RELEASE configuration -- To view, visit

[kudu-CR] catalog manager: don't log deleted tables/tablets at startup

2017-08-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: catalog_manager: don't log deleted tables/tablets at startup .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/7826/2//COMMIT_MSG Commit

  1   2   >