You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Grant Henke (Code Review)" <ge...@cloudera.org> on 2019/04/09 20:44:30 UTC
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12970
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
[docs] Adjust latency and bandwidth limitations
Adjusts the documentation for latency and bandwidth
limitations to be a bit more conservative. Additionaly,
removes any reference to DC/AZ deployments given
we are primarily concered about latency and bandwidth
as the concrete limitations.
Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
---
M docs/known_issues.adoc
1 file changed, 2 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/70/12970/1
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Mike Percy (Code Review)" <ge...@cloudera.org>.
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 22:57:20 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Hello Will Berkeley, Mike Percy, Alexey Serbin, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12970
to look at the new patch set (#3).
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
[docs] Adjust latency and bandwidth limitations
Adjusts the documentation for latency and bandwidth
limitations to be a bit more conservative. Additionally,
removes any reference to DC/AZ deployments given
we are primarily concerned about latency and bandwidth
as the concrete limitations.
Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
---
M docs/known_issues.adoc
1 file changed, 6 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/70/12970/3
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 3: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 21:44:21 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 2: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 18:16:37 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has removed a vote on this change.
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Hello Will Berkeley, Mike Percy, Alexey Serbin, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12970
to look at the new patch set (#2).
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
[docs] Adjust latency and bandwidth limitations
Adjusts the documentation for latency and bandwidth
limitations to be a bit more conservative. Additionally,
removes any reference to DC/AZ deployments given
we are primarily concerned about latency and bandwidth
as the concrete limitations.
Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
---
M docs/known_issues.adoc
1 file changed, 2 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/70/12970/2
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 1:
(2 comments)
Thanks for putting up this update!
http://gerrit.cloudera.org:8080/#/c/12970/1//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/12970/1//COMMIT_MSG@10
PS1, Line 10: Additionaly
Additionally
http://gerrit.cloudera.org:8080/#/c/12970/1//COMMIT_MSG@12
PS1, Line 12: concered
concerned
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 16:23:04 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 2: Code-Review+2
(1 comment)
This looks good to me, but maybe Adar or Will might have some additional thoughts/ideas. Probably, it's worth to wait for their approval as well before submitting this patch.
http://gerrit.cloudera.org:8080/#/c/12970/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1:
> I removed any reference to multi AZ/DC because I didn't want to indicate or
Yep, I agree with Grant that mentioning multi-AZ/DC implicitly endorses the feature that we should play more with before endorsing. Also, if mentioning multi-AZ/DC, probably that would entail more details on how to make the deployment HA-hardened: how many locations to define, how to distribute masters and tablet servers among locations, etc.
I think at this point the purpose of providing this information is more about warning users on very basic restrictions we are aware of once the note of non-supported multi-AZ/multi-DC deployments has been removed from this document.
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 18:49:18 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 3: Code-Review+2
(1 comment)
http://gerrit.cloudera.org:8080/#/c/12970/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1:
> Grant and I talked about this offline.
SGTM, thanks for the clarification.
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 22:34:12 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/12970/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1:
Should we provide at least some context for this? I mean, it's useful advice in the abstract, but perhaps we could hint that this should be particularly relevant to users trying to deploy clusters that span multiple locations?
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 17:04:32 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 3:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/12970/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1:
> Yep, I agree with Grant that mentioning multi-AZ/DC implicitly endorses the
Grant and I talked about this offline.
I'm trying to walk a fine line between being sensitive to the ongoing multi-AZ/DC debate while still offering some useful context. I think briefly mentioning "different locations" (where 'location' is exactly the noun exposed by new the location awareness feature) does just that. It doesn't imply multi-AZ or multi-DC. In fact, you could argue that measuring latency and bandwidth between different _racks_ is a useful thing to do; this calls that out without muddying the waters with multi-AZ or multi-DC.
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 18:55:26 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 1:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/12970/1//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/12970/1//COMMIT_MSG@10
PS1, Line 10: Additionaly
> Additionally
Done
http://gerrit.cloudera.org:8080/#/c/12970/1//COMMIT_MSG@12
PS1, Line 12: concered
> concerned
Done
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 18:15:13 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
[docs] Adjust latency and bandwidth limitations
Adjusts the documentation for latency and bandwidth
limitations to be a bit more conservative. Additionally,
removes any reference to DC/AZ deployments given
we are primarily concerned about latency and bandwidth
as the concrete limitations.
Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Reviewed-on: http://gerrit.cloudera.org:8080/12970
Reviewed-by: Will Berkeley <wd...@gmail.com>
Tested-by: Grant Henke <gr...@apache.org>
Reviewed-by: Alexey Serbin <as...@cloudera.com>
Reviewed-by: Mike Percy <mp...@apache.org>
---
M docs/known_issues.adoc
1 file changed, 6 insertions(+), 4 deletions(-)
Approvals:
Will Berkeley: Looks good to me, approved
Grant Henke: Verified
Alexey Serbin: Looks good to me, approved
Mike Percy: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/12970/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1:
> Should we provide at least some context for this? I mean, it's useful advic
What additional context are you thinking specifically? I left out multi-AZ/DC because it's not clear we want to encourage or discourage certain deployment types.
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 18:13:57 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Will Berkeley (Code Review)" <ge...@cloudera.org>.
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 19:05:14 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/12970/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1:
> What additional context are you thinking specifically? I left out multi-AZ/
Basically, anyone using the location awareness feature to deploy a cross-location cluster should be aware of this, and maybe make an effort to measure latency/bandwidth between locations before deploying.
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 18:30:55 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/12970/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1:
> Yep, I agree with Grant that mentioning multi-AZ/DC implicitly endorses the
Update to use the word "location" and encourage proactive testing based on Adar's feedback. That is generic enough it should be okay.
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 18:52:31 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] Adjust latency and bandwidth limitations
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12970 )
Change subject: [docs] Adjust latency and bandwidth limitations
......................................................................
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/12970/1/docs/known_issues.adoc
File docs/known_issues.adoc:
PS1:
> Basically, anyone using the location awareness feature to deploy a cross-lo
I removed any reference to multi AZ/DC because I didn't want to indicate or endorse support for that feature. My original patches and suggested changes included that context but it opened long and detailed discussion about remaining work to encourage deployments like that.
For that reason, this patch contains only concrete quantifiable limitations based on existing knowledge/deployments to warn/guide anyone who wants a more "creative" deployment in the future.
--
To view, visit http://gerrit.cloudera.org:8080/12970
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I42fe8a65779c3f5ad366403366e534fa28713a76
Gerrit-Change-Number: 12970
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Wed, 10 Apr 2019 18:34:49 +0000
Gerrit-HasComments: Yes