[kudu-CR] [docs] Refresh and augment the known issues
Jean-Daniel Cryans has abandoned this change. Change subject: [docs] Refresh and augment the known issues .. Abandoned I always do this thing where I push against the master branch. -- To view, visit http://gerrit.cloudera.org:8080/6784 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: abandon Gerrit-Change-Id: Iba37e3b6a0d76d46be14cef350a217dd7e71cd7f Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] [docs] Refresh and augment the known issues
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6784 to review the following change. Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augment the known issues We've learned a lot about Kudu since people have started using it. I've gathered in this patch what I think should be the new recommendations we make to users. Cherry-picked from master but had a conflict in developing.adoc since master now allows renaming primary keys. Change-Id: Iba37e3b6a0d76d46be14cef350a217dd7e71cd7f --- M docs/developing.adoc M docs/installation.adoc M docs/known_issues.adoc M docs/kudu_impala_integration.adoc 4 files changed, 106 insertions(+), 20 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/84/6784/1 -- To view, visit http://gerrit.cloudera.org:8080/6784 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iba37e3b6a0d76d46be14cef350a217dd7e71cd7f Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Dan Burkert
[kudu-CR] [docs] Refresh and augment the known issues
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augment the known issues We've learned a lot about Kudu since people have started using it. I've gathered in this patch what I think should be the new recommendations we make to users. Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Reviewed-on: http://gerrit.cloudera.org:8080/6699 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert--- M docs/developing.adoc M docs/installation.adoc M docs/known_issues.adoc M docs/kudu_impala_integration.adoc 4 files changed, 107 insertions(+), 21 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 6 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] [docs] Refresh and augment the known issues
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6699 to look at the new patch set (#5). Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augment the known issues We've learned a lot about Kudu since people have started using it. I've gathered in this patch what I think should be the new recommendations we make to users. Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d --- M docs/developing.adoc M docs/installation.adoc M docs/known_issues.adoc M docs/kudu_impala_integration.adoc 4 files changed, 107 insertions(+), 21 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/6699/5 -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] [docs] Refresh and augment the known issues
Dan Burkert has posted comments on this change. Change subject: [docs] Refresh and augment the known issues .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6699/4/docs/developing.adoc File docs/developing.adoc: Line 167: - `NULL`, `NOT NULL`, `<>`, `OR`, `LIKE`, and `IN` predicates are not pushed to This needs to be updated, `IN`, `NOT NULL`, `NULL`, can be taken out. And some `LIKE` queries are also pushed down now (LIKE "foo%") -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] [docs] Refresh and augment the known issues
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6699 to look at the new patch set (#4). Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augment the known issues We've learned a lot about Kudu since people have started using it. I've gathered in this patch what I think should be the new recommendations we make to users. Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d --- M docs/developing.adoc M docs/installation.adoc M docs/known_issues.adoc M docs/kudu_impala_integration.adoc 4 files changed, 103 insertions(+), 18 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/6699/4 -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] [docs] Refresh and augment the known issues
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Refresh and augment the known issues .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/6699/3/docs/developing.adoc File docs/developing.adoc: Line 171: - Kudu only supports temporary tables in SparkSQL. > this is kind of odd wording. Done http://gerrit.cloudera.org:8080/#/c/6699/3/docs/known_issues.adoc File docs/known_issues.adoc: PS3, Line 129: tservers > we don't abbreviate like thisin the docs Done PS3, Line 129: , > punctuation Done PS3, Line 130: It was also reported that time servers provided my Microsoft : on Azure can get out of sync easily and it’s best configure your own NTP servers > the "voice" of this seems a bit odd/informal. Done PS3, Line 133: * Kudu cannot use Chrony as a time synchronization provider, ntp must be used. > we've seen conflicting reports of this. We need to get to the bottom of it. Done -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] [docs] Refresh and augment the known issues
Todd Lipcon has posted comments on this change. Change subject: [docs] Refresh and augment the known issues .. Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/6699/3/docs/developing.adoc File docs/developing.adoc: Line 171: - Kudu only supports temporary tables in SparkSQL. this is kind of odd wording. I would say something like: - Kudu tables may only be registered as temporary tables in SparkSQL. Kudu tables may not be queried using HiveContext. http://gerrit.cloudera.org:8080/#/c/6699/3/docs/known_issues.adoc File docs/known_issues.adoc: PS3, Line 113: Tablet servers need at least 4GB of memory I'd say "at least 4GB of memory is recommended" or something. You can certainly develop locally using smaller. Not sure the best way to express it. PS3, Line 129: , punctuation PS3, Line 129: tservers we don't abbreviate like thisin the docs PS3, Line 130: It was also reported that time servers provided my Microsoft : on Azure can get out of sync easily and it’s best configure your own NTP servers the "voice" of this seems a bit odd/informal. PS3, Line 133: * Kudu cannot use Chrony as a time synchronization provider, ntp must be used. we've seen conflicting reports of this. We need to get to the bottom of it. Maybe better to say that Kudu releases are only tested using NTP and YMMV with Chrony or somesuch? -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] [docs] Refresh and augment the known issues
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6699 to look at the new patch set (#3). Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augment the known issues We've learned a lot about Kudu since people have started using it. I've gathered in this patch what I think should be the new recommendations we make to users. Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d --- M docs/developing.adoc M docs/installation.adoc M docs/known_issues.adoc M docs/kudu_impala_integration.adoc 4 files changed, 101 insertions(+), 18 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/6699/3 -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] [docs] Refresh and augment the known issues
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] Refresh and augment the known issues .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/6699/2/docs/known_issues.adoc File docs/known_issues.adoc: PS2, Line 135: Supported > This seems out of place in the Known Issues doc - There is a requirements s Well it's "Known Issues and Limitations", so you're limited to those. But I agree it should only be in one place. PS2, Line 168: == Spark > The Admin topic also has a list of limitations for Spark integration - coul I'll add that there. PS2, Line 172: Impala > Same comment as Spark - Consider maintaining in 1 place rather than two. Done -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] [docs] Refresh and augment the known issues
Ambreen Kazi has posted comments on this change. Change subject: [docs] Refresh and augment the known issues .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/6699/2/docs/known_issues.adoc File docs/known_issues.adoc: PS2, Line 135: Supported This seems out of place in the Known Issues doc - There is a requirements section at the top of the installation doc that already lists the supported OSs. The supported filesystems could be added there. PS2, Line 168: == Spark The Admin topic also has a list of limitations for Spark integration - could you take a look and make sure they're still applicable? We should also maintain this information in 1 topic, rather than two. Perhaps move the content from the Admin topic to this one and just link back from the Admin topic to this section (or vice versa). https://kudu.apache.org/docs/developing.html#_spark_integration_known_issues_and_limitations PS2, Line 172: Impala Same comment as Spark - Consider maintaining in 1 place rather than two. https://kudu.apache.org/docs/kudu_impala_integration.html#_known_issues_and_limitations -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR] [docs] Refresh and augment the known issues
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6699 to look at the new patch set (#2). Change subject: [docs] Refresh and augment the known issues .. [docs] Refresh and augment the known issues We've learned a lot about Kudu since people have started using it. I've gathered in this patch what I think should be the new recommendations we make to users. Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d --- M docs/known_issues.adoc 1 file changed, 111 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/6699/2 -- To view, visit http://gerrit.cloudera.org:8080/6699 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I5d8e817a402f419aeb5ed9d700a8207ad9f91e4d Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel CryansGerrit-Reviewer: Ambreen Kazi Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon