You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Andrew Wong (Code Review)" <ge...@cloudera.org> on 2020/10/19 23:21:46 UTC
[kudu-CR](gh-pages) site: add an ecosystem page
Hello Attila Bukor, Grant Henke,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/16615
to review the following change.
Change subject: site: add an ecosystem page
......................................................................
site: add an ecosystem page
I mostly pasted blurbs I found on each components' websites.
A staged version can be seen here:
https://github.com/andrwng/kudu/blob/ecosystem/ecosystem.md
Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
---
M community.md
A ecosystem.md
2 files changed, 156 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/15/16615/1
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/16615/1//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/16615/1//COMMIT_MSG@12
PS1, Line 12: https://github.com/andrwng/kudu/blob/ecosystem/ecosystem.md
> I don't feel strongly about it, it's just a bit weird for me to see this in
Ok, I'll keep it in then. I prefer these links being in the commit message since I find it more accessible for reviewers than burying the link in gerrit comments, especially when there are many revisions.
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Comment-Date: Wed, 21 Oct 2020 21:11:30 +0000
Gerrit-HasComments: Yes
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
Patch Set 2:
(9 comments)
http://gerrit.cloudera.org:8080/#/c/16615/1//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/16615/1//COMMIT_MSG@12
PS1, Line 12: https://github.com/andrwng/kudu/blob/ecosystem/ecosystem.md
> I think this needs to be added only as a comment on the gerrit review, it i
I like keeping them in since I typically don't touch old branches, but if you feel strongly about it i can remove it.
http://gerrit.cloudera.org:8080/#/c/16615/1/community.md
File community.md:
http://gerrit.cloudera.org:8080/#/c/16615/1/community.md@57
PS1, Line 57: * [**Apache Kudu Ecosystem**](/ecosystem.html)
> I think this should also be listed under the community menu
Done
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md
File ecosystem.md:
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@8
PS1, Line 8: ## Apache Kudu Ecosystem
> I searched for more integrations we may want to include and found these:
Done
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@15
PS1, Line 15: ### Frequently used
> I think Apache NiFi belongs in this list.
Done
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@20
PS1, Line 20: * [Apache Impala](#apache-impala)
> Hive may actually be not that common yet.
Done
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@23
PS1, Line 23: * [Presto](#presto)
> I think it's called Presto, only their URL is prestodb.com as presto.com se
Done
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@25
PS1, Line 25: SQL
> Should we add a clients section that links to our C++, Java, and Python doc
If you feel strongly about it, but it seems somewhat tangential to the ecosystem.
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@75
PS1, Line 75: #### [Pandas](https://pandas.pydata.org/)
> This is supported via the Python client right? I think its worth noting tha
Done
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@136
PS1, Line 136:
> Do we also want to list distribution and support offerings?
Done
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Comment-Date: Wed, 21 Oct 2020 19:30:00 +0000
Gerrit-HasComments: Yes
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
Patch Set 2: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Comment-Date: Thu, 22 Oct 2020 18:02:47 +0000
Gerrit-HasComments: No
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Comment-Date: Thu, 22 Oct 2020 16:58:19 +0000
Gerrit-HasComments: No
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
site: add an ecosystem page
I mostly pasted blurbs I found on each components' websites.
A staged version can be seen here:
https://github.com/andrwng/kudu/blob/ecosystem/ecosystem.md
Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Reviewed-on: http://gerrit.cloudera.org:8080/16615
Reviewed-by: Grant Henke <gr...@apache.org>
Reviewed-by: Attila Bukor <ab...@apache.org>
Tested-by: Andrew Wong <aw...@cloudera.com>
---
M _includes/top_common.html
M community.md
A ecosystem.md
3 files changed, 188 insertions(+), 0 deletions(-)
Approvals:
Grant Henke: Looks good to me, approved
Attila Bukor: Looks good to me, approved
Andrew Wong: Verified
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Comment-Date: Thu, 22 Oct 2020 16:55:20 +0000
Gerrit-HasComments: No
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
Patch Set 1:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/16615/1//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/16615/1//COMMIT_MSG@12
PS1, Line 12: https://github.com/andrwng/kudu/blob/ecosystem/ecosystem.md
I think this needs to be added only as a comment on the gerrit review, it isn't really useful in the commit message once it's committed, and the link will be broken too once you delete the branch.
http://gerrit.cloudera.org:8080/#/c/16615/1/community.md
File community.md:
http://gerrit.cloudera.org:8080/#/c/16615/1/community.md@57
PS1, Line 57: * [**Apache Kudu Ecosystem**](/ecosystem.html)
I think this should also be listed under the community menu
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md
File ecosystem.md:
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@23
PS1, Line 23: * [Presto DB](#presto-db)
I think it's called Presto, only their URL is prestodb.com as presto.com seems to be taken by someone else.
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Comment-Date: Tue, 20 Oct 2020 07:40:26 +0000
Gerrit-HasComments: Yes
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/16615/1//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/16615/1//COMMIT_MSG@12
PS1, Line 12: https://github.com/andrwng/kudu/blob/ecosystem/ecosystem.md
> I like keeping them in since I typically don't touch old branches, but if y
I don't feel strongly about it, it's just a bit weird for me to see this in a commit message.
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Comment-Date: Wed, 21 Oct 2020 21:02:36 +0000
Gerrit-HasComments: Yes
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Hello Attila Bukor, Grant Henke,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16615
to look at the new patch set (#2).
Change subject: site: add an ecosystem page
......................................................................
site: add an ecosystem page
I mostly pasted blurbs I found on each components' websites.
A staged version can be seen here:
https://github.com/andrwng/kudu/blob/ecosystem/ecosystem.md
Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
---
M _includes/top_common.html
M community.md
A ecosystem.md
3 files changed, 188 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/15/16615/2
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
Patch Set 1:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md
File ecosystem.md:
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@8
PS1, Line 8: ## Apache Kudu Ecosystem
I searched for more integrations we may want to include and found these:
- https://www.zoomdata.com/product/big-data/big-data-analytics-kudu/
- https://docs.wavefront.com/kudu.html
-https://help.talend.com/reader/SuRq3Ek0vdlxbl_OV_wVFQ/TVdA7QTsoiR9zzODVu78wA
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@15
PS1, Line 15: ### Frequently used
I think Apache NiFi belongs in this list.
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@20
PS1, Line 20: * [Apache Hive](#apache-hive)
Hive may actually be not that common yet.
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@25
PS1, Line 25: SQL
Should we add a clients section that links to our C++, Java, and Python docs or source?
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@75
PS1, Line 75: #### [Pandas](https://pandas.pydata.org/)
This is supported via the Python client right? I think its worth noting that we provide the support, not the pandas project.
http://gerrit.cloudera.org:8080/#/c/16615/1/ecosystem.md@136
PS1, Line 136: ### Deployment and orchestration
Do we also want to list distribution and support offerings?
- https://www.cloudera.com/products/open-source/apache-hadoop/apache-kudu.html
- https://www.phdata.io/getting-started-with-kudu/
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Comment-Date: Tue, 20 Oct 2020 18:00:46 +0000
Gerrit-HasComments: Yes
[kudu-CR](gh-pages) site: add an ecosystem page
Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16615 )
Change subject: site: add an ecosystem page
......................................................................
Patch Set 1: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/16615
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib22d09afdf3ae8240f1ff64c7121b5216f6b3345
Gerrit-Change-Number: 16615
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Comment-Date: Tue, 20 Oct 2020 07:41:03 +0000
Gerrit-HasComments: No