[kudu-CR] [docs] updated docs w.r.t. collocation practices

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12997 )

Change subject: [docs] updated docs w.r.t. collocation practices
..


Patch Set 4:

What is the status of this? Is this abandoned?


--
To view, visit http://gerrit.cloudera.org:8080/12997
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I88a38117751cfa436c1fd95598274fb8f01f04ea
Gerrit-Change-Number: 12997
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mitch Barnett 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Sat, 10 Aug 2019 01:01:25 +
Gerrit-HasComments: No

[kudu-CR] docs: add the required config for Sentry

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13902 )

Change subject: docs: add the required config for Sentry
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/13902/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/13902/1//COMMIT_MSG@7
PS1, Line 7: docs: add the required config for Sentry
Could you add a note that this is for upstream only?



--
To view, visit http://gerrit.cloudera.org:8080/13902
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If208579405c11fc25aefbedd3f63ac21afecbddf
Gerrit-Change-Number: 13902
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 23 Jul 2019 22:22:45 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] Fix formatting in administration

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13129 )

Change subject: [docs] Fix formatting in administration
..


Patch Set 1: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/13129
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I189cb7ce3dfd04442452932cd39a83d5041ac42e
Gerrit-Change-Number: 13129
Gerrit-PatchSet: 1
Gerrit-Owner: Attila Bukor 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Fri, 26 Apr 2019 18:02:43 +
Gerrit-HasComments: No

[kudu-CR] KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12493 )

Change subject: KUDU-2508: [DOCS] Recommend nscd even for static name 
resolutions
..


Patch Set 4:

Adar,
Could you do Verified+1 and submit this? Thank you!


--
To view, visit http://gerrit.cloudera.org:8080/12493
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
Gerrit-Change-Number: 12493
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Sat, 16 Feb 2019 00:53:57 +
Gerrit-HasComments: No

[kudu-CR] KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12493 )

Change subject: KUDU-2508: [DOCS] Recommend nscd even for static name 
resolutions
..


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12493/3/docs/troubleshooting.adoc
File docs/troubleshooting.adoc:

http://gerrit.cloudera.org:8080/#/c/12493/3/docs/troubleshooting.adoc@663
PS3, Line 663: nscd
> I think we still want the backticks so it gets formatted as a command. See
Sure??? Done



--
To view, visit http://gerrit.cloudera.org:8080/12493
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
Gerrit-Change-Number: 12493
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Fri, 15 Feb 2019 22:28:33 +
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

[Alex Rodoni (Code Review)]

Hello Will Berkeley, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/12493

to look at the new patch set (#4).

Change subject: KUDU-2508: [DOCS] Recommend nscd even for static name 
resolutions
..

KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
---
M docs/troubleshooting.adoc
1 file changed, 9 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/12493/4
--
To view, visit http://gerrit.cloudera.org:8080/12493
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
Gerrit-Change-Number: 12493
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 

[kudu-CR] KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12493 )

Change subject: KUDU-2508: [DOCS] Recommend nscd even for static name 
resolutions
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc
File docs/troubleshooting.adoc:

http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc@679
PS1, Line 679: - A slow name resolution can be caused heavy system contentions 
when opening the
> Yeah I would just remove it. The only useful information would be the reaso
Done



--
To view, visit http://gerrit.cloudera.org:8080/12493
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
Gerrit-Change-Number: 12493
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Fri, 15 Feb 2019 20:26:39 +
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

[Alex Rodoni (Code Review)]

Hello Will Berkeley, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/12493

to look at the new patch set (#3).

Change subject: KUDU-2508: [DOCS] Recommend nscd even for static name 
resolutions
..

KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
---
M docs/troubleshooting.adoc
1 file changed, 10 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/12493/3
--
To view, visit http://gerrit.cloudera.org:8080/12493
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
Gerrit-Change-Number: 12493
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 

[kudu-CR] KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12493 )

Change subject: KUDU-2508: [DOCS] Recommend nscd even for static name 
resolutions
..


Patch Set 1:

(8 comments)

http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc
File docs/troubleshooting.adoc:

http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc@665
PS1, Line 665: N
> Lower-case n
Done


http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc@665
PS1, Line 665: Nscd
> Since this is a software tool, lowercase `nscd`
Done


http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc@668
PS1, Line 668: yo
> you
Done


http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc@668
PS1, Line 668: yo
> you
Done


http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc@671
PS1, Line 671: For example:
> IMO this reads kind of odd because it's not an example of how to use `nscd`
Attempted


http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc@679
PS1, Line 679: be caused heavy system contentions
> cause heavy system contention
Done


http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc@679
PS1, Line 679: - A slow name resolution can be caused heavy system contentions 
when opening the
> Not really sure what this second bullet is adding. Unlike the first bullet,
Let me know if I should just remove this part.


http://gerrit.cloudera.org:8080/#/c/12493/1/docs/troubleshooting.adoc@680
PS1, Line 680: files
> file
Done



--
To view, visit http://gerrit.cloudera.org:8080/12493
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
Gerrit-Change-Number: 12493
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Fri, 15 Feb 2019 20:17:24 +
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

[Alex Rodoni (Code Review)]

Hello Will Berkeley, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/12493

to look at the new patch set (#2).

Change subject: KUDU-2508: [DOCS] Recommend nscd even for static name 
resolutions
..

KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
---
M docs/troubleshooting.adoc
1 file changed, 12 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/12493/2
--
To view, visit http://gerrit.cloudera.org:8080/12493
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
Gerrit-Change-Number: 12493
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 

[kudu-CR] KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/12493


Change subject: KUDU-2508: [DOCS] Recommend nscd even for static name 
resolutions
..

KUDU-2508: [DOCS] Recommend nscd even for static name resolutions

Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
---
M docs/troubleshooting.adoc
1 file changed, 13 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/12493/1
--
To view, visit http://gerrit.cloudera.org:8080/12493
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I93b192a2c664262477200f34f3819e48bcbbfc26
Gerrit-Change-Number: 12493
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR] [docs] An instruction to add repaired disk back to cluster

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12329 )

Change subject: [docs] An instruction to add repaired disk back to cluster
..


Patch Set 1:

Adar, could you submit this?


--
To view, visit http://gerrit.cloudera.org:8080/12329
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4d268f62455a22899f3bf38a4bf4ad64db048457
Gerrit-Change-Number: 12329
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Fri, 01 Feb 2019 00:24:32 +
Gerrit-HasComments: No

[kudu-CR] [docs] An instruction to add repaired disk back to cluster

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12329 )

Change subject: [docs] An instruction to add repaired disk back to cluster
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12329/1/docs/administration.adoc
File docs/administration.adoc:

http://gerrit.cloudera.org:8080/#/c/12329/1/docs/administration.adoc@1029
PS1, Line 1029:
> Nit: is there an extra empty line here?
Without this extra line, the note gets indented to align the last list item.



--
To view, visit http://gerrit.cloudera.org:8080/12329
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4d268f62455a22899f3bf38a4bf4ad64db048457
Gerrit-Change-Number: 12329
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Fri, 01 Feb 2019 00:09:13 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] An instruction to add repaired disk back to cluster

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/12329


Change subject: [docs] An instruction to add repaired disk back to cluster
..

[docs] An instruction to add repaired disk back to cluster

Change-Id: I4d268f62455a22899f3bf38a4bf4ad64db048457
---
M docs/administration.adoc
1 file changed, 26 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/29/12329/1
--
To view, visit http://gerrit.cloudera.org:8080/12329
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4d268f62455a22899f3bf38a4bf4ad64db048457
Gerrit-Change-Number: 12329
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR] KUDU-1375: [docs] Remove "binaries" in the Build from Source section

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12060 )

Change subject: KUDU-1375: [docs] Remove "binaries" in the Build from Source 
section
..


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12060/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/12060/3//COMMIT_MSG@9
PS3, Line 9: make
> Nit: 'install' would be more appropriate here.
Done



--
To view, visit http://gerrit.cloudera.org:8080/12060
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
Gerrit-Change-Number: 12060
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 11 Dec 2018 20:16:17 +
Gerrit-HasComments: Yes

[kudu-CR] KUDU-1375: [docs] Remove "binaries" in the Build from Source section

[Alex Rodoni (Code Review)]

Hello Kudu Jenkins, Adar Dembo,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/12060

to look at the new patch set (#4).

Change subject: KUDU-1375: [docs] Remove "binaries" in the Build from Source 
section
..

KUDU-1375: [docs] Remove "binaries" in the Build from Source section

- make install does not install the Kudu binaries

Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
---
M docs/installation.adoc
1 file changed, 33 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/60/12060/4
--
To view, visit http://gerrit.cloudera.org:8080/12060
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
Gerrit-Change-Number: 12060
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] KUDU-1375: [docs] Remove "binaries" in the Build from Source section

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12060 )

Change subject: KUDU-1375: [docs] Remove "binaries" in the Build from Source 
section
..


Patch Set 2:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/12060/2/docs/installation.adoc
File docs/installation.adoc:

http://gerrit.cloudera.org:8080/#/c/12060/2/docs/installation.adoc@420
PS2, Line 420: NOTE: 'make install' does not copy Kudu binaries. You need to 
manually copy them
> This text block should be replicated into the other _from_source sections (
Done


http://gerrit.cloudera.org:8080/#/c/12060/2/docs/installation.adoc@424
PS2, Line 424: the your
> Nit: drop one of these.
Done


http://gerrit.cloudera.org:8080/#/c/12060/2/docs/installation.adoc@426
PS2, Line 426: if you use the default installation directory
> I would tie this more directly to DESTDIR from L413-414.
Done



--
To view, visit http://gerrit.cloudera.org:8080/12060
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
Gerrit-Change-Number: 12060
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 11 Dec 2018 00:46:29 +
Gerrit-HasComments: Yes

[kudu-CR] KUDU-1375: [docs] Remove "binaries" in the Build from Source section

[Alex Rodoni (Code Review)]

Hello Kudu Jenkins, Adar Dembo,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/12060

to look at the new patch set (#3).

Change subject: KUDU-1375: [docs] Remove "binaries" in the Build from Source 
section
..

KUDU-1375: [docs] Remove "binaries" in the Build from Source section

- make install does not make the Kudu binaries

Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
---
M docs/installation.adoc
1 file changed, 33 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/60/12060/3
--
To view, visit http://gerrit.cloudera.org:8080/12060
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
Gerrit-Change-Number: 12060
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] KUDU-1375: [docs] Remove "binaries" in the Build from Source section

[Alex Rodoni (Code Review)]

Hello Kudu Jenkins, Adar Dembo,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/12060

to look at the new patch set (#2).

Change subject: KUDU-1375: [docs] Remove "binaries" in the Build from Source 
section
..

KUDU-1375: [docs] Remove "binaries" in the Build from Source section

- make install does not make the Kudu binaries

Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
---
M docs/installation.adoc
1 file changed, 12 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/60/12060/2
--
To view, visit http://gerrit.cloudera.org:8080/12060
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
Gerrit-Change-Number: 12060
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] KUDU-1375: [docs] Remove "binaries" in the Build from Source section

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/12060


Change subject: KUDU-1375: [docs] Remove "binaries" in the Build from Source 
section
..

KUDU-1375: [docs] Remove "binaries" in the Build from Source section

- make install does not make the Kudu binaries

Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
---
M docs/installation.adoc
1 file changed, 3 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/60/12060/1
--
To view, visit http://gerrit.cloudera.org:8080/12060
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5b1e7337cdd9f04eea1c2b4da3d27b9815553121
Gerrit-Change-Number: 12060
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR] docs: add a note about RAID-0

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11580 )

Change subject: docs: add a note about RAID-0
..


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/11580/1/docs/configuration.adoc
File docs/configuration.adoc:

http://gerrit.cloudera.org:8080/#/c/11580/1/docs/configuration.adoc@77
PS1, Line 77:
the ones with


http://gerrit.cloudera.org:8080/#/c/11580/1/docs/configuration.adoc@76
PS1, Line 76: Note that while RAID-0
: devices with multiple drives may perform better than single 
drives, using a
: single RAID-0 device instead of specifying multiple data 
directories may reduce
: the amount of parallelism with which Kudu can operate.
Maybe put this in a separate Note paragraph below?



--
To view, visit http://gerrit.cloudera.org:8080/11580
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb17fae9459be04211bd0a7db6eca8e94e443f71
Gerrit-Change-Number: 11580
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Thu, 04 Oct 2018 19:59:07 +
Gerrit-HasComments: Yes

[kudu-CR] [DOCS] Added a version notice for changing managed Kudu table names in Impala

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11515 )

Change subject: [DOCS] Added a version notice for changing managed Kudu table 
names in Impala
..


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/11515/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11515/2//COMMIT_MSG@7
PS2, Line 7: ]
> nit: an extra closing brace; please remove it
Done


http://gerrit.cloudera.org:8080/#/c/11515/2//COMMIT_MSG@9
PS2, Line 9: With IMPALA-5654, users cannot change kudu.table_name in IMPALA for
   : managed Kudu tables created in Impala.
> It looks like nitpicking, but this sentence contains mention of Impala twic
Done



--
To view, visit http://gerrit.cloudera.org:8080/11515
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If26f103d931dd7ed57b56cf34b0010d14c098928
Gerrit-Change-Number: 11515
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Wed, 26 Sep 2018 21:08:34 +
Gerrit-HasComments: Yes

[kudu-CR] [DOCS] Added a version notice for changing managed Kudu table names in Impala

[Alex Rodoni (Code Review)]

Hello Mike Percy, Alexey Serbin, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/11515

to look at the new patch set (#3).

Change subject: [DOCS] Added a version notice for changing managed Kudu table 
names in Impala
..

[DOCS] Added a version notice for changing managed Kudu table names in Impala

With IMPALA-5654, users can no longer change kudu.table_name property
for managed Kudu tables in Impala.

Change-Id: If26f103d931dd7ed57b56cf34b0010d14c098928
---
M docs/kudu_impala_integration.adoc
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/15/11515/3
--
To view, visit http://gerrit.cloudera.org:8080/11515
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If26f103d931dd7ed57b56cf34b0010d14c098928
Gerrit-Change-Number: 11515
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 

[kudu-CR] ][DOCS] Added a version notice for changing managed Kudu table names in Impala

[Alex Rodoni (Code Review)]

Hello Mike Percy, Alexey Serbin, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/11515

to look at the new patch set (#2).

Change subject: ][DOCS] Added a version notice for changing managed Kudu table 
names in Impala
..

][DOCS] Added a version notice for changing managed Kudu table names in Impala

With IMPALA-5654, users cannot change kudu.table_name in IMPALA for
managed Kudu tables created in Impala.

Change-Id: If26f103d931dd7ed57b56cf34b0010d14c098928
---
M docs/kudu_impala_integration.adoc
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/15/11515/2
--
To view, visit http://gerrit.cloudera.org:8080/11515
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If26f103d931dd7ed57b56cf34b0010d14c098928
Gerrit-Change-Number: 11515
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 

[kudu-CR] ][DOCS] Added a version notice for changing managed Kudu table names in Impala

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11515 )

Change subject: ][DOCS] Added a version notice for changing managed Kudu table 
names in Impala
..


Patch Set 1:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/11515/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11515/1//COMMIT_MSG@9
PS1, Line 9: With IMPALA-5654, users cannot change kudu.table_name in IMPALA.
> I think this is too generic and incorrect: for external tables, they still
Done


http://gerrit.cloudera.org:8080/#/c/11515/1/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:

http://gerrit.cloudera.org:8080/#/c/11515/1/docs/kudu_impala_integration.adoc@687
PS1, Line 687:  if the table is an internal table
> The enclosed section is named
Done


http://gerrit.cloudera.org:8080/#/c/11515/1/docs/kudu_impala_integration.adoc@697
PS1, Line 697: In Impala 2.11 and lower, if another application has renamed a 
Kudu table under
 : Impala, it is possible to re-map an external table to point to a 
different Kudu
 : table name.
> I meant:  ... That it's impossible to re-map an external table to a differe
Version note removed


http://gerrit.cloudera.org:8080/#/c/11515/1/docs/kudu_impala_integration.adoc@697
PS1, Line 697: In Impala 2.11 and lower, if another application has renamed a 
Kudu table under
 : Impala, it is possible to re-map an external table to point to a 
different Kudu
 : table name.
> I think this became confusing.  What's the extra 'In Impala 2.11 and lower
Done



--
To view, visit http://gerrit.cloudera.org:8080/11515
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If26f103d931dd7ed57b56cf34b0010d14c098928
Gerrit-Change-Number: 11515
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Wed, 26 Sep 2018 00:59:00 +
Gerrit-HasComments: Yes

[kudu-CR] [DOCS] Added a version notice for changing kudu table names in Impala

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11515


Change subject: [DOCS] Added a version notice for changing kudu table names in 
Impala
..

[DOCS] Added a version notice for changing kudu table names in Impala

With IMPALA-5654, users cannot change kudu.table_name in IMPALA.

Change-Id: If26f103d931dd7ed57b56cf34b0010d14c098928
---
M docs/kudu_impala_integration.adoc
1 file changed, 5 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/15/11515/1
--
To view, visit http://gerrit.cloudera.org:8080/11515
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If26f103d931dd7ed57b56cf34b0010d14c098928
Gerrit-Change-Number: 11515
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR] [DOCS] Fixed a broken link in administration.adoc

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11066


Change subject: [DOCS] Fixed a broken link in administration.adoc
..

[DOCS] Fixed a broken link in administration.adoc

Change-Id: I7ca94cc24b3038659b89c62d1e1ac18e74acf65a
---
M docs/administration.adoc
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/66/11066/1
--
To view, visit http://gerrit.cloudera.org:8080/11066
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7ca94cc24b3038659b89c62d1e1ac18e74acf65a
Gerrit-Change-Number: 11066
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR] [DOCS] Updated the max and recommended numbers of tablets per server

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/10969


Change subject: [DOCS] Updated the max and recommended numbers of tablets per 
server
..

[DOCS] Updated the max and recommended numbers of tablets per server

Change-Id: Ida16e45477bd18206f25db8cc2bf890742d4ec63
---
M docs/known_issues.adoc
1 file changed, 4 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/10969/1
--
To view, visit http://gerrit.cloudera.org:8080/10969
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ida16e45477bd18206f25db8cc2bf890742d4ec63
Gerrit-Change-Number: 10969
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR] [DOCS] Removed KUDU-1942 and KUDU-2032 from Known Issues List

[Alex Rodoni (Code Review)]

Hello Kudu Jenkins, Todd Lipcon,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/10953

to look at the new patch set (#2).

Change subject: [DOCS] Removed KUDU-1942 and KUDU-2032 from Known Issues List
..

[DOCS] Removed KUDU-1942 and KUDU-2032 from Known Issues List

Change-Id: I84bdf0193df08241ea632e1b1b51476bba40403c
---
M docs/known_issues.adoc
1 file changed, 0 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/10953/2
--
To view, visit http://gerrit.cloudera.org:8080/10953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I84bdf0193df08241ea632e1b1b51476bba40403c
Gerrit-Change-Number: 10953
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 

[kudu-CR] [DOCS] Removed KUDU-1942 and KUDU-2032 from Known Issues List

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10953 )

Change subject: [DOCS] Removed KUDU-1942 and KUDU-2032 from Known Issues List
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10953/1/docs/known_issues.adoc
File docs/known_issues.adoc:

http://gerrit.cloudera.org:8080/#/c/10953/1/docs/known_issues.adoc@186
PS1, Line 186:   was fixed in Kudu 1.5.
> Usually we just remove known issues after they are fixed. The old/archived
Done



--
To view, visit http://gerrit.cloudera.org:8080/10953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I84bdf0193df08241ea632e1b1b51476bba40403c
Gerrit-Change-Number: 10953
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Mon, 16 Jul 2018 21:31:29 +
Gerrit-HasComments: Yes

[kudu-CR] [DOCS] Marked KUDU-1942 and KUDU-2032 as fixed

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/10953


Change subject: [DOCS] Marked KUDU-1942 and KUDU-2032 as fixed
..

[DOCS] Marked KUDU-1942 and KUDU-2032 as fixed

Change-Id: I84bdf0193df08241ea632e1b1b51476bba40403c
---
M docs/known_issues.adoc
1 file changed, 4 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/10953/1
--
To view, visit http://gerrit.cloudera.org:8080/10953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I84bdf0193df08241ea632e1b1b51476bba40403c
Gerrit-Change-Number: 10953
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR] [DOCS] Fixed a link in non-covering partition doc

[Alex Rodoni (Code Review)]

Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Andrew Wong,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/10891

to look at the new patch set (#3).

Change subject: [DOCS] Fixed a link in non-covering partition doc
..

[DOCS] Fixed a link in non-covering partition doc

Change-Id: Ice86fb23a15679e08b18ec56437bb545daaa80cc
---
M docs/kudu_impala_integration.adoc
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/10891/3
--
To view, visit http://gerrit.cloudera.org:8080/10891
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ice86fb23a15679e08b18ec56437bb545daaa80cc
Gerrit-Change-Number: 10891
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 

[kudu-CR] [DOCS] Fixed a link in non-covering partition doc

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10891 )

Change subject: [DOCS] Fixed a link in non-covering partition doc
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10891/2/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:

http://gerrit.cloudera.org:8080/#/c/10891/2/docs/kudu_impala_integration.adoc@469
PS2, Line 469: in
> nit: I'm not sure, but to my taste it sounds better if replacing 'in' with
Done



--
To view, visit http://gerrit.cloudera.org:8080/10891
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ice86fb23a15679e08b18ec56437bb545daaa80cc
Gerrit-Change-Number: 10891
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Tue, 10 Jul 2018 02:29:22 +
Gerrit-HasComments: Yes

[kudu-CR] [DOCS] Fixed a link in non-covering partition doc

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10891 )

Change subject: [DOCS] Fixed a link in non-covering partition doc
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10891/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/10891/1//COMMIT_MSG@7
PS1, Line 7: coverting
> nit: covering
Done



--
To view, visit http://gerrit.cloudera.org:8080/10891
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ice86fb23a15679e08b18ec56437bb545daaa80cc
Gerrit-Change-Number: 10891
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Mon, 09 Jul 2018 23:38:30 +
Gerrit-HasComments: Yes

[kudu-CR] [DOCS] Fixed a link in non-covering partition doc

[Alex Rodoni (Code Review)]

Hello Mike Percy, Kudu Jenkins, Andrew Wong,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/10891

to look at the new patch set (#2).

Change subject: [DOCS] Fixed a link in non-covering partition doc
..

[DOCS] Fixed a link in non-covering partition doc

Change-Id: Ice86fb23a15679e08b18ec56437bb545daaa80cc
---
M docs/kudu_impala_integration.adoc
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/10891/2
--
To view, visit http://gerrit.cloudera.org:8080/10891
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ice86fb23a15679e08b18ec56437bb545daaa80cc
Gerrit-Change-Number: 10891
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 

[kudu-CR] [DOCS] Fixed a link in non-coverting partition doc

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/10891


Change subject: [DOCS] Fixed a link in non-coverting partition doc
..

[DOCS] Fixed a link in non-coverting partition doc

Change-Id: Ice86fb23a15679e08b18ec56437bb545daaa80cc
---
M docs/kudu_impala_integration.adoc
1 file changed, 1 insertion(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/10891/1
--
To view, visit http://gerrit.cloudera.org:8080/10891
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ice86fb23a15679e08b18ec56437bb545daaa80cc
Gerrit-Change-Number: 10891
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR](gh-pages) Update website with doc improvements from master

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10730 )

Change subject: Update website with doc improvements from master
..


Patch Set 2: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/10730
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic846d497793bd16022cad7f0b185979bd68639a6
Gerrit-Change-Number: 10730
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Mike Percy 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Fri, 15 Jun 2018 18:12:03 +
Gerrit-HasComments: No

[kudu-CR] Adding a subsection to NTP troubleshooting section to cover frequent cases of NTP crashes when the Kudu cluster is deployed on Microsoft Azure

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8960 )

Change subject: Adding a subsection to NTP troubleshooting section to cover 
frequent cases of NTP crashes when the Kudu cluster is deployed on Microsoft 
Azure
..


Patch Set 3:

(3 comments)

And please remove any trailing spaces.

http://gerrit.cloudera.org:8080/#/c/8960/3/docs/troubleshooting.adoc
File docs/troubleshooting.adoc:

http://gerrit.cloudera.org:8080/#/c/8960/3/docs/troubleshooting.adoc@125
PS3, Line 125:
Remove


http://gerrit.cloudera.org:8080/#/c/8960/3/docs/troubleshooting.adoc@128
PS3, Line 128: kerberized
the Kerberized


http://gerrit.cloudera.org:8080/#/c/8960/3/docs/troubleshooting.adoc@137
PS3, Line 137: Following frequent root causes can help troubleshoot the issue.
The following are some of the root causes frequently encountered. Review the 
issues to see if you can apply any to your troubleshooting.



--
To view, visit http://gerrit.cloudera.org:8080/8960
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5d638aa26137d3a795fce1d0f3f744d4969bdf22
Gerrit-Change-Number: 8960
Gerrit-PatchSet: 3
Gerrit-Owner: Mahdi Askari 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mahdi Askari 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Wed, 30 May 2018 18:56:10 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] Add short troubleshooting note about nscd

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10498 )

Change subject: [docs] Add short troubleshooting note about nscd
..


Patch Set 1:

Does this apply to all versions?


--
To view, visit http://gerrit.cloudera.org:8080/10498
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace99c33d9161a0878c7c7570313ca5f895a7a9
Gerrit-Change-Number: 10498
Gerrit-PatchSet: 1
Gerrit-Owner: Will Berkeley 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Thu, 24 May 2018 19:18:15 +
Gerrit-HasComments: No

[kudu-CR] [DOCS] Added an info on creating a table in Impala with a custom replication factor

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10411 )

Change subject: [DOCS] Added an info on creating a table in Impala with a 
custom replication factor
..


Patch Set 2:

Hi Mike,
Could you review and approve this change?


--
To view, visit http://gerrit.cloudera.org:8080/10411
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If1c9ee97497dd62ccc8769428435ba6d5adddae9
Gerrit-Change-Number: 10411
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Thu, 17 May 2018 01:30:53 +
Gerrit-HasComments: No

[kudu-CR] [DOCS] Added an info on creating a table in Impala with a custom replication factor

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10411 )

Change subject: [DOCS] Added an info on creating a table in Impala with a 
custom replication factor
..


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/10411/1/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:

http://gerrit.cloudera.org:8080/#/c/10411/1/docs/kudu_impala_integration.adoc@147
PS1, Line 147: To change
> Instead of To change, I think we should say to specify
Done


http://gerrit.cloudera.org:8080/#/c/10411/1/docs/kudu_impala_integration.adoc@148
PS1, Line 148: the `TBLPROPERTIES` in the  statement
> adding a TBLPROPERTIES clause to the CREATE TABLE statement
Done


http://gerrit.cloudera.org:8080/#/c/10411/1/docs/kudu_impala_integration.adoc@149
PS1, Line 149: you want to use:
> also note: replication factor must be an odd number, changing this table pr
Done



--
To view, visit http://gerrit.cloudera.org:8080/10411
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If1c9ee97497dd62ccc8769428435ba6d5adddae9
Gerrit-Change-Number: 10411
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Tue, 15 May 2018 20:33:51 +
Gerrit-HasComments: Yes

[kudu-CR] [DOCS] Added an info on creating a table in Impala with a custom replication factor

[Alex Rodoni (Code Review)]

Hello Mike Percy, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/10411

to look at the new patch set (#2).

Change subject: [DOCS] Added an info on creating a table in Impala with a 
custom replication factor
..

[DOCS] Added an info on creating a table in Impala with a custom replication 
factor

Change-Id: If1c9ee97497dd62ccc8769428435ba6d5adddae9
---
M docs/kudu_impala_integration.adoc
1 file changed, 15 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/11/10411/2
--
To view, visit http://gerrit.cloudera.org:8080/10411
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If1c9ee97497dd62ccc8769428435ba6d5adddae9
Gerrit-Change-Number: 10411
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 

[kudu-CR] [DOCS] Added an info on creating a table in Impala with a custom replication factor

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/10411


Change subject: [DOCS] Added an info on creating a table in Impala with a 
custom replication factor
..

[DOCS] Added an info on creating a table in Impala with a custom replication 
factor

Change-Id: If1c9ee97497dd62ccc8769428435ba6d5adddae9
---
M docs/kudu_impala_integration.adoc
1 file changed, 10 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/11/10411/1
--
To view, visit http://gerrit.cloudera.org:8080/10411
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If1c9ee97497dd62ccc8769428435ba6d5adddae9
Gerrit-Change-Number: 10411
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR] Fix table formatting in full data dirs docs

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10211 )

Change subject: Fix table formatting in full data dirs docs
..


Patch Set 1:

The table looks correct now!

I cannot do +2 for Kudu, though.


--
To view, visit http://gerrit.cloudera.org:8080/10211
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I098c64c3241e5fc4184b102e10ef619feb7de30a
Gerrit-Change-Number: 10211
Gerrit-PatchSet: 1
Gerrit-Owner: Will Berkeley 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Wed, 25 Apr 2018 22:47:31 +
Gerrit-HasComments: No

[kudu-CR] Fix table formatting in full data dirs docs

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10211 )

Change subject: Fix table formatting in full data dirs docs
..


Patch Set 1: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/10211
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I098c64c3241e5fc4184b102e10ef619feb7de30a
Gerrit-Change-Number: 10211
Gerrit-PatchSet: 1
Gerrit-Owner: Will Berkeley 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Wed, 25 Apr 2018 22:16:49 +
Gerrit-HasComments: No

[kudu-CR] [docs] Add docs on full data dirs

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9967 )

Change subject: [docs] Add docs on full data dirs
..


Patch Set 6:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/9967/6/docs/administration.adoc
File docs/administration.adoc:

http://gerrit.cloudera.org:8080/#/c/9967/6/docs/administration.adoc@828
PS6, Line 828: ,<
Remove this.



--
To view, visit http://gerrit.cloudera.org:8080/9967
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If1081cab4c84789d29a0ccdccfd11c190b6164cf
Gerrit-Change-Number: 9967
Gerrit-PatchSet: 6
Gerrit-Owner: Will Berkeley 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Wed, 25 Apr 2018 22:13:36 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] Add docs on full data dirs

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9967 )

Change subject: [docs] Add docs on full data dirs
..


Patch Set 5:

The table format needs to be fixed. It is rendering with 3 columns and 
misplacing cells.


--
To view, visit http://gerrit.cloudera.org:8080/9967
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If1081cab4c84789d29a0ccdccfd11c190b6164cf
Gerrit-Change-Number: 9967
Gerrit-PatchSet: 5
Gerrit-Owner: Will Berkeley 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Wed, 25 Apr 2018 21:30:43 +
Gerrit-HasComments: No

[kudu-CR] docs: update docs for update dirs tool

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9110 )

Change subject: docs: update docs for update_dirs tool
..


Patch Set 3: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/9110
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic3c139326aee35f72495a1cb1aaf8df1d58776cb
Gerrit-Change-Number: 9110
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Wong 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Sat, 03 Mar 2018 00:07:55 +
Gerrit-HasComments: No

[kudu-CR] docs: add warning to wait between FS rebuilds

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9279 )

Change subject: docs: add warning to wait between FS rebuilds
..


Patch Set 4: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/9279
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5659c2ba05a0a6b9905213c5df6ba0dcb371f312
Gerrit-Change-Number: 9279
Gerrit-PatchSet: 4
Gerrit-Owner: Andrew Wong 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Sat, 03 Mar 2018 00:05:19 +
Gerrit-HasComments: No

[kudu-CR] [doc] Document the new decimal column type

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9432 )

Change subject: [doc] Document the new decimal column type
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/9432/2/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/9432/2/docs/release_notes.adoc@53
PS2, Line 53: a
the



--
To view, visit http://gerrit.cloudera.org:8080/9432
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9489613d35daad708648ea04d49e472d3149b33d
Gerrit-Change-Number: 9432
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Fri, 23 Feb 2018 22:14:17 +
Gerrit-HasComments: Yes

[kudu-CR] docs: update docs for update dirs tool

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9110 )

Change subject: docs: update docs for update_dirs tool
..


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/9110/2/docs/administration.adoc
File docs/administration.adoc:

http://gerrit.cloudera.org:8080/#/c/9110/2/docs/administration.adoc@797
PS2, Line 797: In the event that critical files are lost (i.e. WALs or 
tablet-specific
 : metadata), to ensure correctness, all Kudu directories on the 
server must be deleted
 : and rebuilt,
In the event that critical files are lost, i.e. WALS or tablet-specific 
metadata, all Kudu directories on the server must be deleted and rebuilt to 
ensure correctness.


http://gerrit.cloudera.org:8080/#/c/9110/2/docs/administration.adoc@799
PS2, Line 799: destroying
who destroys? The customer or Kudu? Start a new sentence that "... will 
destroy..."



--
To view, visit http://gerrit.cloudera.org:8080/9110
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic3c139326aee35f72495a1cb1aaf8df1d58776cb
Gerrit-Change-Number: 9110
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Wed, 07 Feb 2018 20:15:39 +
Gerrit-HasComments: Yes

[kudu-CR] docs: improvements to transaction semantics

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9235 )

Change subject: docs: improvements to transaction semantics
..


Patch Set 1: Code-Review+1

Which version of Kudu does this apply?


--
To view, visit http://gerrit.cloudera.org:8080/9235
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I23a2751923a4214f52e6cdb233f7e3aeee207da2
Gerrit-Change-Number: 9235
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: David Ribeiro Alves 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Wed, 07 Feb 2018 01:46:20 +
Gerrit-HasComments: No

[kudu-CR] [docs] Add scaling guide

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8842 )

Change subject: [docs] Add scaling guide
..


Patch Set 3:

(7 comments)

http://gerrit.cloudera.org:8080/#/c/8842/3/docs/scaling_guide.adoc
File docs/scaling_guide.adoc:

http://gerrit.cloudera.org:8080/#/c/8842/3/docs/scaling_guide.adoc@15
PS3, Line 15: Please
Remove


http://gerrit.cloudera.org:8080/#/c/8842/3/docs/scaling_guide.adoc@20
PS3, Line 20: on use
a line break?


http://gerrit.cloudera.org:8080/#/c/8842/3/docs/scaling_guide.adoc@28
PS3, Line 28:  a
A


http://gerrit.cloudera.org:8080/#/c/8842/3/docs/scaling_guide.adoc@31
PS3, Line 31: a
A


http://gerrit.cloudera.org:8080/#/c/8842/3/docs/scaling_guide.adoc@32
PS3, Line 32: say
for example,


http://gerrit.cloudera.org:8080/#/c/8842/3/docs/scaling_guide.adoc@35
PS3, Line 35: the
The


http://gerrit.cloudera.org:8080/#/c/8842/3/docs/scaling_guide.adoc@165
PS3, Line 165: limit
OS limit



--
To view, visit http://gerrit.cloudera.org:8080/8842
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I38d8999addc41fe0b726342a27dbba199ddf7dd2
Gerrit-Change-Number: 8842
Gerrit-PatchSet: 3
Gerrit-Owner: Will Berkeley 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Tue, 06 Feb 2018 23:48:50 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] Document how to recover from a majority failed tablet

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8402 )

Change subject: [docs] Document how to recover from a majority failed tablet
..


Patch Set 7:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/8402/7/docs/administration.adoc
File docs/administration.adoc:

http://gerrit.cloudera.org:8080/#/c/8402/7/docs/administration.adoc@811
PS7, Line 811: majority
is there a quantifiable definition of "majority"?


http://gerrit.cloudera.org:8080/#/c/8402/7/docs/administration.adoc@813
PS7, Line 813: and so
potentially resulting in permanent data loss.


http://gerrit.cloudera.org:8080/#/c/8402/7/docs/administration.adoc@840
PS7, Line 840: ,
remove ","


http://gerrit.cloudera.org:8080/#/c/8402/7/docs/administration.adoc@844
PS7, Line 844: ,
Remove ,


http://gerrit.cloudera.org:8080/#/c/8402/7/docs/administration.adoc@844
PS7, Line 844: those
Those



--
To view, visit http://gerrit.cloudera.org:8080/8402
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic6326f65d029a1cd75e487b16ce5be4baea2f215
Gerrit-Change-Number: 8402
Gerrit-PatchSet: 7
Gerrit-Owner: Will Berkeley 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Tue, 06 Feb 2018 23:37:08 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] Added recommendation to compress PK for backfill inserts

[Alex Rodoni (Code Review)]

Hello Dan Burkert, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/9185

to look at the new patch set (#4).

Change subject: [docs] Added recommendation to compress PK for backfill inserts
..

[docs] Added recommendation to compress PK for backfill inserts

Change-Id: I698d954265b4171e4d1bb7e01e286d0d489f1ec7
---
M docs/schema_design.adoc
1 file changed, 64 insertions(+), 18 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/85/9185/4
--
To view, visit http://gerrit.cloudera.org:8080/9185
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I698d954265b4171e4d1bb7e01e286d0d489f1ec7
Gerrit-Change-Number: 9185
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Kudu Jenkins

[kudu-CR] [docs] Added recommendation to compress PK for backfill inserts

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9185 )

Change subject: [docs] Added recommendation to compress PK for backfill inserts
..


Patch Set 4:

Fixed a typo in the Backfill section heading.


--
To view, visit http://gerrit.cloudera.org:8080/9185
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I698d954265b4171e4d1bb7e01e286d0d489f1ec7
Gerrit-Change-Number: 9185
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Sat, 03 Feb 2018 00:46:00 +
Gerrit-HasComments: No

[kudu-CR] [docs] Added recommendation to compress PK for backfill inserts

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/9185


Change subject: [docs] Added recommendation to compress PK for backfill inserts
..

[docs] Added recommendation to compress PK for backfill inserts

Change-Id: I698d954265b4171e4d1bb7e01e286d0d489f1ec7
---
M docs/schema_design.adoc
1 file changed, 64 insertions(+), 18 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/85/9185/2
--
To view, visit http://gerrit.cloudera.org:8080/9185
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I698d954265b4171e4d1bb7e01e286d0d489f1ec7
Gerrit-Change-Number: 9185
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Dan Burkert 

[kudu-CR] docs: fix number of number masters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9091 )

Change subject: docs: fix number of number masters
..


Patch Set 3:

Which version of Kudu is this for? Current 1.6?

Is there a JIRA id for this?


--
To view, visit http://gerrit.cloudera.org:8080/9091
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic24ae1cbcd59534273adba48b20a2418651b19a4
Gerrit-Change-Number: 9091
Gerrit-PatchSet: 3
Gerrit-Owner: Attila Bukor 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Wed, 24 Jan 2018 00:48:38 +
Gerrit-HasComments: No

[kudu-CR] docs: fix number of number masters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9091 )

Change subject: docs: fix number of number masters
..


Patch Set 3:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/9091/3/docs/installation.adoc
File docs/installation.adoc:

http://gerrit.cloudera.org:8080/#/c/9091/3/docs/installation.adoc@45
PS3, Line 45: should always be
must be


http://gerrit.cloudera.org:8080/#/c/9091/3/docs/installation.adoc@49
PS3, Line 49: Two
For example, having two


http://gerrit.cloudera.org:8080/#/c/9091/3/docs/installation.adoc@50
PS3, Line 50: any
Not needed



--
To view, visit http://gerrit.cloudera.org:8080/9091
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic24ae1cbcd59534273adba48b20a2418651b19a4
Gerrit-Change-Number: 9091
Gerrit-PatchSet: 3
Gerrit-Owner: Attila Bukor 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Tue, 23 Jan 2018 21:30:37 +
Gerrit-HasComments: Yes

[kudu-CR] docs: update docs for metadata dir

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9068 )

Change subject: docs: update docs for metadata dir
..


Patch Set 2: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/9068
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I333d32080106cd83b92ad2060f3239b9c44d201b
Gerrit-Change-Number: 9068
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Fri, 19 Jan 2018 01:29:21 +
Gerrit-HasComments: No

[kudu-CR] docs: update docs for metadata dir

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9068 )

Change subject: docs: update docs for metadata dir
..


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/9068/1/docs/configuration.adoc
File docs/configuration.adoc:

http://gerrit.cloudera.org:8080/#/c/9068/1/docs/configuration.adoc@66
PS1, Line 66: drives (i.e. with high bandwidth
: and low latency, e.g. solid-state drives)
drives with high bandwidth and low latency, e.g. solid-state drives.


http://gerrit.cloudera.org:8080/#/c/9068/1/docs/configuration.adoc@72
PS1, Line 72: multiple
multiple values



--
To view, visit http://gerrit.cloudera.org:8080/9068
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I333d32080106cd83b92ad2060f3239b9c44d201b
Gerrit-Change-Number: 9068
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Thu, 18 Jan 2018 23:54:57 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] Added a section for Kudu clients connecting to secure clusters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8953 )

Change subject: [docs] Added a section for Kudu clients connecting to secure 
clusters
..


Patch Set 8: -Code-Review

This is ready for review.


--
To view, visit http://gerrit.cloudera.org:8080/8953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
Gerrit-Change-Number: 8953
Gerrit-PatchSet: 8
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Thu, 18 Jan 2018 23:16:55 +
Gerrit-HasComments: No

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8948 )

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..


Patch Set 15: -Code-Review

This is ready for review.


--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 15
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Thu, 18 Jan 2018 23:16:39 +
Gerrit-HasComments: No

[kudu-CR] [docs] Removed the note about KUDU-1626

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9020 )

Change subject: [docs] Removed the note about KUDU-1626
..


Patch Set 1: -Code-Review

This is ready for review.


--
To view, visit http://gerrit.cloudera.org:8080/9020
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7b5e127cd9d1e2bda60a20de62fafd95e5f93e3
Gerrit-Change-Number: 9020
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Thu, 18 Jan 2018 23:16:21 +
Gerrit-HasComments: No

[kudu-CR] [docs] Added a section for Kudu clients connecting to secure clusters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8953 )

Change subject: [docs] Added a section for Kudu clients connecting to secure 
clusters
..


Patch Set 8: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/8953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
Gerrit-Change-Number: 8953
Gerrit-PatchSet: 8
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Thu, 18 Jan 2018 00:21:52 +
Gerrit-HasComments: No

[kudu-CR] [docs] Removed the note about KUDU-1626

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9020 )

Change subject: [docs] Removed the note about KUDU-1626
..


Patch Set 1: Code-Review+1

Could you please review and approve the change?


--
To view, visit http://gerrit.cloudera.org:8080/9020
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7b5e127cd9d1e2bda60a20de62fafd95e5f93e3
Gerrit-Change-Number: 9020
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Thu, 18 Jan 2018 00:21:15 +
Gerrit-HasComments: No

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8948 )

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..


Patch Set 15: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 15
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Thu, 18 Jan 2018 00:21:39 +
Gerrit-HasComments: No

[kudu-CR] [docs] Removed the note about KUDU-1626

[Alex Rodoni (Code Review)]

Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/9020


Change subject: [docs] Removed the note about KUDU-1626
..

[docs] Removed the note about KUDU-1626

Change-Id: Ia7b5e127cd9d1e2bda60a20de62fafd95e5f93e3
---
M docs/schema_design.adoc
1 file changed, 0 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/20/9020/1
--
To view, visit http://gerrit.cloudera.org:8080/9020
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia7b5e127cd9d1e2bda60a20de62fafd95e5f93e3
Gerrit-Change-Number: 9020
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8948

to look at the new patch set (#14).

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..

[docs] Added steps to update HMS after migrating to multiple Kudu masters

Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
---
M docs/administration.adoc
1 file changed, 36 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/8948/14
--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 14
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8948 )

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..


Patch Set 13:

> (1 comment)

 > (1 comment)

Done - I think


--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 13
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Fri, 12 Jan 2018 19:55:04 +
Gerrit-HasComments: No

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8948 )

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..


Patch Set 13:

> Uploaded patch set 13.

Changed "Impala-shell" to "impala-shell" with code font.


--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 13
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Thu, 11 Jan 2018 23:29:32 +
Gerrit-HasComments: No

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8948

to look at the new patch set (#13).

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..

[docs] Added steps to update HMS after migrating to multiple Kudu masters

Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
---
M docs/administration.adoc
A docs/patch
2 files changed, 60 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/8948/13
--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 13
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8948

to look at the new patch set (#12).

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..

[docs] Added steps to update HMS after migrating to multiple Kudu masters

Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
---
M docs/administration.adoc
1 file changed, 36 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/8948/12
--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 12
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8948 )

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..


Patch Set 10:

(1 comment)

> (1 comment)

DONE

http://gerrit.cloudera.org:8080/#/c/8948/8/docs/security.adoc
File docs/security.adoc:

http://gerrit.cloudera.org:8080/#/c/8948/8/docs/security.adoc@102
PS8, Line 102: == Scalability
> it seems this change snuck into the wrong git commit?
Yes



--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 10
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Thu, 11 Jan 2018 21:55:48 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8948

to look at the new patch set (#11).

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..

[docs] Added steps to update HMS after migrating to multiple Kudu masters

Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
---
M docs/administration.adoc
1 file changed, 36 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/8948/11
--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 11
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8948

to look at the new patch set (#10).

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..

[docs] Added steps to update HMS after migrating to multiple Kudu masters

Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
---
M docs/administration.adoc
1 file changed, 36 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/8948/10
--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 10
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 

[kudu-CR] [docs] Added a section for Kudu clients connecting to secure clusters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8953 )

Change subject: [docs] Added a section for Kudu clients connecting to secure 
clusters
..


Patch Set 1:

(4 comments)

> (4 comments)

done

http://gerrit.cloudera.org:8080/#/c/8953/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/8953/3//COMMIT_MSG@7
PS3, Line 7: Kerberose Kudu
> ?
Done


http://gerrit.cloudera.org:8080/#/c/8953/5//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/8953/5//COMMIT_MSG@7
PS5, Line 7: .
> nit: don't end the first line in a commit message with a period.
Done


http://gerrit.cloudera.org:8080/#/c/8953/1/docs/security.adoc
File docs/security.adoc:

http://gerrit.cloudera.org:8080/#/c/8953/1/docs/security.adoc@102
PS1, Line 102: Server
> I think a better title would be 'Client Authentication to Secure Kudu Clust
Done


http://gerrit.cloudera.org:8080/#/c/8953/1/docs/security.adoc@105
PS1, Line 105: ith
> with the
Done



--
To view, visit http://gerrit.cloudera.org:8080/8953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
Gerrit-Change-Number: 8953
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Thu, 11 Jan 2018 19:38:32 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8948 )

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..


Patch Set 8:

(2 comments)

> (3 comments)

http://gerrit.cloudera.org:8080/#/c/8948/8/docs/administration.adoc
File docs/administration.adoc:

http://gerrit.cloudera.org:8080/#/c/8948/8/docs/administration.adoc@263
PS8, Line 263: statement
> maybe it's just me, but seems like this isn't super clear since this statem
DONE


http://gerrit.cloudera.org:8080/#/c/8948/8/docs/administration.adoc@380
PS8, Line 380: update the HMS database manually.
 : * The following is an example SQL statement:
> per above, I think we should be explicit that this statement must be run ag
DONE



--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 8
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Thu, 11 Jan 2018 19:05:37 +
Gerrit-HasComments: Yes

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8948

to look at the new patch set (#8).

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..

[docs] Added steps to update HMS after migrating to multiple Kudu masters

Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
---
M docs/administration.adoc
M docs/security.adoc
2 files changed, 48 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/8948/8
--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 8
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8948

to look at the new patch set (#7).

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..

[docs] Added steps to update HMS after migrating to multiple Kudu masters

Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
---
M docs/administration.adoc
1 file changed, 35 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/8948/7
--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 7
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 

[kudu-CR] [docs] Added a section for Kudu clients connecting to secure clusters

[Alex Rodoni (Code Review)]

Hello Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8953

to look at the new patch set (#8).

Change subject: [docs] Added a section for Kudu clients connecting to secure 
clusters
..

[docs] Added a section for Kudu clients connecting to secure clusters

Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
---
M docs/security.adoc
1 file changed, 13 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/8953/8
--
To view, visit http://gerrit.cloudera.org:8080/8953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
Gerrit-Change-Number: 8953
Gerrit-PatchSet: 8
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins

[kudu-CR] [docs] Added additional steps for updating HMS when migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8953

to look at the new patch set (#7).

Change subject: [docs] Added additional steps for updating HMS when migrating 
to multiple Kudu masters
..

[docs] Added additional steps for updating HMS when migrating to multiple Kudu 
masters

Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
---
M docs/security.adoc
1 file changed, 13 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/8953/7
--
To view, visit http://gerrit.cloudera.org:8080/8953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
Gerrit-Change-Number: 8953
Gerrit-PatchSet: 7
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8948

to look at the new patch set (#5).

Change subject: [docs] Added steps to update HMS after migrating to multiple 
Kudu masters
..

[docs] Added steps to update HMS after migrating to multiple Kudu masters

Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
---
M docs/administration.adoc
1 file changed, 23 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/8948/5
--
To view, visit http://gerrit.cloudera.org:8080/8948
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iab3999c9e581ed3591b220c08491cdae867c91db
Gerrit-Change-Number: 8948
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall 

[kudu-CR] [docs] Added additional steps for migrating to multiple Kudu masters

[Alex Rodoni (Code Review)]

Hello Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8953

to look at the new patch set (#6).

Change subject: [docs] Added additional steps for migrating to multiple Kudu 
masters
..

[docs] Added additional steps for migrating to multiple Kudu masters

Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
---
M docs/administration.adoc
M docs/security.adoc
2 files changed, 33 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/8953/6
--
To view, visit http://gerrit.cloudera.org:8080/8953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
Gerrit-Change-Number: 8953
Gerrit-PatchSet: 6
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins

[kudu-CR] [doc] Added a section for Kudu clients connecting to secure clusters.

[Alex Rodoni (Code Review)]

Hello Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8953

to look at the new patch set (#5).

Change subject: [doc] Added a section for Kudu clients connecting to secure 
clusters.
..

[doc] Added a section for Kudu clients connecting to secure clusters.

Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
---
M docs/security.adoc
1 file changed, 15 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/53/8953/5
--
To view, visit http://gerrit.cloudera.org:8080/8953
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iabee52216b4db0e18e5f693375f0a3e0f0a8a164
Gerrit-Change-Number: 8953
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Jean-Daniel Cryans 
Gerrit-Reviewer: Kudu Jenkins