You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Alex Rodoni (Code Review)" <ge...@cloudera.org> on 2018/07/30 18:53:08 UTC

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

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


Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................

[DOCS] Dedicated coordinators and executors explained

Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
---
M docs/impala.ditamap
A docs/topics/impala_dedicated_coordinator.xml
M docs/topics/impala_scalability.xml
3 files changed, 630 insertions(+), 114 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/80/11080/1
-- 
To view, visit http://gerrit.cloudera.org:8080/11080
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Alex Rodoni (Code Review)" <ge...@cloudera.org>.
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11080/3/docs/impala_keydefs.ditamap
File docs/impala_keydefs.ditamap:

http://gerrit.cloudera.org:8080/#/c/11080/3/docs/impala_keydefs.ditamap@10960
PS3, Line 10960: impala_dedicated_coordinatori.xml
> A typo here which caused the merge to fail.
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fw...@cloudera.com>
Gerrit-Reviewer: Greg Rahn <gr...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Fri, 03 Aug 2018 17:03:49 +0000
Gerrit-HasComments: Yes

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Alex Rodoni (Code Review)" <ge...@cloudera.org>.
Hello Alan Choi, Philip Zeyliger, Tim Armstrong, 

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

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

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

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................

[DOCS] Dedicated coordinators and executors explained

Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
---
M docs/impala.ditamap
M docs/impala_keydefs.ditamap
A docs/topics/impala_dedicated_coordinator.xml
M docs/topics/impala_scalability.xml
4 files changed, 631 insertions(+), 115 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/80/11080/2
-- 
To view, visit http://gerrit.cloudera.org:8080/11080
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 2: Verified-1

Build Failed 

https://jenkins.impala.io/job/gerrit-docs-auto-test/18/ : Doc tests failed. See linked job for details on the failure.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 30 Jul 2018 22:42:28 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Tim Armstrong (Code Review)" <ge...@cloudera.org>.
Tim Armstrong has removed a vote on this change.

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Removed Verified-1 by Impala Public Jenkins <im...@cloudera.com>
-- 
To view, visit http://gerrit.cloudera.org:8080/11080
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 1:

Build Started https://jenkins.impala.io/job/gerrit-docs-auto-test/17/ 

Testing docs change - this change appears to modify docs/ and no code. This is experimental - please report any issues to tarmstrong@cloudera.com or on this JIRA: IMPALA-7317


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 30 Jul 2018 22:14:37 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 3:

Build Started https://jenkins.impala.io/job/gerrit-docs-auto-test/20/ 

Testing docs change - this change appears to modify docs/ and no code. This is experimental - please report any issues to tarmstrong@cloudera.com or on this JIRA: IMPALA-7317


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 30 Jul 2018 23:40:32 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 4:

Build Started https://jenkins.impala.io/job/gerrit-docs-auto-test/25/ 

Testing docs change - this change appears to modify docs/ and no code. This is experimental - please report any issues to tarmstrong@cloudera.com or on this JIRA: IMPALA-7317


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fw...@cloudera.com>
Gerrit-Reviewer: Greg Rahn <gr...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Fri, 03 Aug 2018 17:02:20 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Fredy Wijaya (Code Review)" <ge...@cloudera.org>.
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11080/3/docs/impala_keydefs.ditamap
File docs/impala_keydefs.ditamap:

http://gerrit.cloudera.org:8080/#/c/11080/3/docs/impala_keydefs.ditamap@10960
PS3, Line 10960: impala_dedicated_coordinatori.xml
A typo here which caused the merge to fail.

[gen-list] [DOTX008E][ERROR] File 'file:/home/ubuntu/Impala/docs/topics/impala_dedicated_coordinatori.xml' does not exist or cannot be loaded.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fw...@cloudera.com>
Gerrit-Reviewer: Greg Rahn <gr...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Fri, 03 Aug 2018 16:49:19 +0000
Gerrit-HasComments: Yes

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Tim Armstrong (Code Review)" <ge...@cloudera.org>.
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 4: Code-Review+2


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fw...@cloudera.com>
Gerrit-Reviewer: Greg Rahn <gr...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Peter Ebert
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 06 Aug 2018 15:59:17 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Alex Rodoni (Code Review)" <ge...@cloudera.org>.
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 2:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/11080/2/docs/topics/impala_dedicated_coordinator.xml
File docs/topics/impala_dedicated_coordinator.xml:

http://gerrit.cloudera.org:8080/#/c/11080/2/docs/topics/impala_dedicated_coordinator.xml@161
PS2, Line 161: caches
> typo: cache
Done


http://gerrit.cloudera.org:8080/#/c/11080/2/docs/topics/impala_dedicated_coordinator.xml@201
PS2, Line 201: diagonalisability
> typo: diagnosability
Done


http://gerrit.cloudera.org:8080/#/c/11080/2/docs/topics/impala_dedicated_coordinator.xml@363
PS2, Line 363: ata Node
> I saw DataNode elsewhere. Either is fine for me.
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 30 Jul 2018 23:41:18 +0000
Gerrit-HasComments: Yes

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Alex Rodoni (Code Review)" <ge...@cloudera.org>.
Alex Rodoni has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................

[DOCS] Dedicated coordinators and executors explained

Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Reviewed-on: http://gerrit.cloudera.org:8080/11080
Tested-by: Impala Public Jenkins <im...@cloudera.com>
Reviewed-by: Tim Armstrong <ta...@cloudera.com>
---
M docs/impala.ditamap
M docs/impala_keydefs.ditamap
A docs/topics/impala_dedicated_coordinator.xml
M docs/topics/impala_scalability.xml
4 files changed, 631 insertions(+), 115 deletions(-)

Approvals:
  Impala Public Jenkins: Verified
  Tim Armstrong: Looks good to me, approved

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fw...@cloudera.com>
Gerrit-Reviewer: Greg Rahn <gr...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Peter Ebert
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 3: Verified-1

Build Failed 

https://jenkins.impala.io/job/gerrit-docs-auto-test/20/ : Doc tests failed. See linked job for details on the failure.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 30 Jul 2018 23:54:25 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Alex Rodoni (Code Review)" <ge...@cloudera.org>.
Hello Greg Rahn, Fredy Wijaya, Alan Choi, Philip Zeyliger, Tim Armstrong, Impala Public Jenkins, 

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

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

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

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................

[DOCS] Dedicated coordinators and executors explained

Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
---
M docs/impala.ditamap
M docs/impala_keydefs.ditamap
A docs/topics/impala_dedicated_coordinator.xml
M docs/topics/impala_scalability.xml
4 files changed, 631 insertions(+), 115 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/80/11080/4
-- 
To view, visit http://gerrit.cloudera.org:8080/11080
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fw...@cloudera.com>
Gerrit-Reviewer: Greg Rahn <gr...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Tim Armstrong (Code Review)" <ge...@cloudera.org>.
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 2: Code-Review+1

(3 comments)

This will be really valuable, the previous documentation was a bit lacking.

http://gerrit.cloudera.org:8080/#/c/11080/2/docs/topics/impala_dedicated_coordinator.xml
File docs/topics/impala_dedicated_coordinator.xml:

http://gerrit.cloudera.org:8080/#/c/11080/2/docs/topics/impala_dedicated_coordinator.xml@161
PS2, Line 161: caches
typo: cache


http://gerrit.cloudera.org:8080/#/c/11080/2/docs/topics/impala_dedicated_coordinator.xml@201
PS2, Line 201: diagonalisability
typo: diagnosability


http://gerrit.cloudera.org:8080/#/c/11080/2/docs/topics/impala_dedicated_coordinator.xml@363
PS2, Line 363: ata Node
I saw DataNode elsewhere. Either is fine for me.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 30 Jul 2018 23:03:59 +0000
Gerrit-HasComments: Yes

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 2:

Build Started https://jenkins.impala.io/job/gerrit-docs-auto-test/18/ 

Testing docs change - this change appears to modify docs/ and no code. This is experimental - please report any issues to tarmstrong@cloudera.com or on this JIRA: IMPALA-7317


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 30 Jul 2018 22:14:38 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Alex Rodoni (Code Review)" <ge...@cloudera.org>.
Hello Alan Choi, Philip Zeyliger, Tim Armstrong, Impala Public Jenkins, 

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

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

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

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................

[DOCS] Dedicated coordinators and executors explained

Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
---
M docs/impala.ditamap
M docs/impala_keydefs.ditamap
A docs/topics/impala_dedicated_coordinator.xml
M docs/topics/impala_scalability.xml
4 files changed, 631 insertions(+), 115 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/80/11080/3
-- 
To view, visit http://gerrit.cloudera.org:8080/11080
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 1: Verified+1

Build Successful 

https://jenkins.impala.io/job/gerrit-docs-auto-test/17/ : Doc tests passed.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Mon, 30 Jul 2018 22:42:52 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Dedicated coordinators and executors explained

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11080 )

Change subject: [DOCS] Dedicated coordinators and executors explained
......................................................................


Patch Set 4: Verified+1

Build Successful 

https://jenkins.impala.io/job/gerrit-docs-auto-test/25/ : Doc tests passed.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic0038eb40951ddf619ea6470a7363122635391d8
Gerrit-Change-Number: 11080
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Alan Choi <al...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <ar...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fw...@cloudera.com>
Gerrit-Reviewer: Greg Rahn <gr...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <ph...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-Comment-Date: Fri, 03 Aug 2018 17:20:43 +0000
Gerrit-HasComments: No