You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Jordan Birdsell (Code Review)" <ge...@cloudera.org> on 2018/09/11 01:31:39 UTC

[kudu-CR](gh-pages) [blog] Data Pipelines Simplified with Kudu

Jordan Birdsell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11417


Change subject: [blog] Data Pipelines Simplified with Kudu
......................................................................

[blog] Data Pipelines Simplified with Kudu

Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
---
A _posts/2018-09-11-simplified-pipelines-with-kudu.md
1 file changed, 44 insertions(+), 0 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newchange
Gerrit-Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
Gerrit-Change-Number: 11417
Gerrit-PatchSet: 1
Gerrit-Owner: Jordan Birdsell <jt...@apache.org>

[kudu-CR](gh-pages) [blog] Data Pipelines Simplified with Kudu

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

Change subject: [blog] Data Pipelines Simplified with Kudu
......................................................................


Patch Set 1: Verified+1

(1 comment)

verified that it's rendering correctly

http://gerrit.cloudera.org:8080/#/c/11417/1/_posts/2018-09-11-simplified-pipelines-with-kudu.md
File _posts/2018-09-11-simplified-pipelines-with-kudu.md:

http://gerrit.cloudera.org:8080/#/c/11417/1/_posts/2018-09-11-simplified-pipelines-with-kudu.md@38
PS1, Line 38: value add
I think this word is hyphenated, or maybe it should be replaced by some other word like 'enhanced' to sound less a sales pitch-y.



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
Gerrit-Change-Number: 11417
Gerrit-PatchSet: 1
Gerrit-Owner: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Brock Noland <br...@phdata.io>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Tue, 11 Sep 2018 08:26:43 +0000
Gerrit-HasComments: Yes

[kudu-CR](gh-pages) [blog] Data Pipelines Simplified with Kudu

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

Change subject: [blog] Data Pipelines Simplified with Kudu
......................................................................


Patch Set 2: Verified+1 Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
Gerrit-Change-Number: 11417
Gerrit-PatchSet: 2
Gerrit-Owner: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Brock Noland <br...@phdata.io>
Gerrit-Reviewer: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Tue, 11 Sep 2018 15:44:30 +0000
Gerrit-HasComments: No

[kudu-CR](gh-pages) [blog] Data Pipelines Simplified with Kudu

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

Change subject: [blog] Data Pipelines Simplified with Kudu
......................................................................


Patch Set 2:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/11417/1/_posts/2018-09-11-simplified-pipelines-with-kudu.md
File _posts/2018-09-11-simplified-pipelines-with-kudu.md:

http://gerrit.cloudera.org:8080/#/c/11417/1/_posts/2018-09-11-simplified-pipelines-with-kudu.md@38
PS1, Line 38: ficant 
> I think this word is hyphenated, or maybe it should be replaced by some oth
Done


http://gerrit.cloudera.org:8080/#/c/11417/1/_posts/2018-09-11-simplified-pipelines-with-kudu.md@42
PS1, Line 42: e of 
> caches ?
Done


http://gerrit.cloudera.org:8080/#/c/11417/1/_posts/2018-09-11-simplified-pipelines-with-kudu.md@42
PS1, Line 42: changes, and re
> updates/deletes ?  Maybe, rephrase it to '... of merging in changes, and ..
Done



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
Gerrit-Change-Number: 11417
Gerrit-PatchSet: 2
Gerrit-Owner: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Brock Noland <br...@phdata.io>
Gerrit-Reviewer: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Tue, 11 Sep 2018 13:25:13 +0000
Gerrit-HasComments: Yes

[kudu-CR](gh-pages) [blog] Data Pipelines Simplified with Kudu

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

Change subject: [blog] Data Pipelines Simplified with Kudu
......................................................................


Patch Set 1:

(2 comments)

A couple of nits.

http://gerrit.cloudera.org:8080/#/c/11417/1/_posts/2018-09-11-simplified-pipelines-with-kudu.md
File _posts/2018-09-11-simplified-pipelines-with-kudu.md:

http://gerrit.cloudera.org:8080/#/c/11417/1/_posts/2018-09-11-simplified-pipelines-with-kudu.md@42
PS1, Line 42: cache
caches ?


http://gerrit.cloudera.org:8080/#/c/11417/1/_posts/2018-09-11-simplified-pipelines-with-kudu.md@42
PS1, Line 42: updates/updates
updates/deletes ?  Maybe, rephrase it to '... of merging in changes, and ...'



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
Gerrit-Change-Number: 11417
Gerrit-PatchSet: 1
Gerrit-Owner: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Brock Noland <br...@phdata.io>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Tue, 11 Sep 2018 02:01:51 +0000
Gerrit-HasComments: Yes

[kudu-CR](gh-pages) [blog] Data Pipelines Simplified with Kudu

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

Change subject: [blog] Data Pipelines Simplified with Kudu
......................................................................

[blog] Data Pipelines Simplified with Kudu

Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
Reviewed-on: http://gerrit.cloudera.org:8080/11417
Reviewed-by: Attila Bukor <ab...@apache.org>
Tested-by: Attila Bukor <ab...@apache.org>
---
A _posts/2018-09-11-simplified-pipelines-with-kudu.md
1 file changed, 44 insertions(+), 0 deletions(-)

Approvals:
  Attila Bukor: Looks good to me, approved; Verified

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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: merged
Gerrit-Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
Gerrit-Change-Number: 11417
Gerrit-PatchSet: 3
Gerrit-Owner: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Brock Noland <br...@phdata.io>
Gerrit-Reviewer: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>

[kudu-CR](gh-pages) [blog] Data Pipelines Simplified with Kudu

Posted by "Jordan Birdsell (Code Review)" <ge...@cloudera.org>.
Hello Will Berkeley, Alexey Serbin, Attila Bukor, Todd Lipcon, Brock Noland, 

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

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

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

Change subject: [blog] Data Pipelines Simplified with Kudu
......................................................................

[blog] Data Pipelines Simplified with Kudu

Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
---
A _posts/2018-09-11-simplified-pipelines-with-kudu.md
1 file changed, 44 insertions(+), 0 deletions(-)


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I222d2462da86c3aad3fa9afd71f686faaa9aa025
Gerrit-Change-Number: 11417
Gerrit-PatchSet: 2
Gerrit-Owner: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Brock Noland <br...@phdata.io>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>