You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Attila Bukor (Code Review)" <ge...@cloudera.org> on 2018/08/22 19:03:33 UTC

[kudu-CR](gh-pages) Add archived releases section

Hello Mike Percy, Grant Henke, Todd Lipcon,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: Add archived releases section
......................................................................

Add archived releases section

Change-Id: I671f58c8c67604876573f31936a5dc764302098b
---
M releases/index.md
1 file changed, 4 insertions(+), 1 deletion(-)



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newchange
Gerrit-Change-Id: I671f58c8c67604876573f31936a5dc764302098b
Gerrit-Change-Number: 11295
Gerrit-PatchSet: 1
Gerrit-Owner: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>

[kudu-CR](gh-pages) Add archived releases section

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

Change subject: Add archived releases section
......................................................................


Patch Set 3:

> Uploaded patch set 3.

Anyone knows what the LATREL/PREVREL tags do and if they're ok like this? They were added with 60be4d7 by Mike and they show up on the site without being replaced by anything.


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I671f58c8c67604876573f31936a5dc764302098b
Gerrit-Change-Number: 11295
Gerrit-PatchSet: 3
Gerrit-Owner: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Comment-Date: Wed, 22 Aug 2018 19:08:44 +0000
Gerrit-HasComments: No

[kudu-CR](gh-pages) Add archived releases section

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

Change subject: Add archived releases section
......................................................................


Patch Set 3:

> Anyone knows what the LATREL/PREVREL tags do and if they're ok like this? They were added with 60be4d7 by Mike and they show up on the site without being replaced by anything.

See https://gerrit.cloudera.org/c/8800/


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I671f58c8c67604876573f31936a5dc764302098b
Gerrit-Change-Number: 11295
Gerrit-PatchSet: 3
Gerrit-Owner: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Comment-Date: Fri, 12 Oct 2018 22:00:12 +0000
Gerrit-HasComments: No

[kudu-CR](gh-pages) Add archived releases section

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

Change subject: Add archived releases section
......................................................................


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: abandon
Gerrit-Change-Id: I671f58c8c67604876573f31936a5dc764302098b
Gerrit-Change-Number: 11295
Gerrit-PatchSet: 3
Gerrit-Owner: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>

[kudu-CR](gh-pages) Add archived releases section

Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
Hello Mike Percy, Grant Henke, Todd Lipcon, 

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

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

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

Change subject: Add archived releases section
......................................................................

Add archived releases section

Change-Id: I671f58c8c67604876573f31936a5dc764302098b
---
M releases/index.md
1 file changed, 5 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/95/11295/3
-- 
To view, visit http://gerrit.cloudera.org:8080/11295
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I671f58c8c67604876573f31936a5dc764302098b
Gerrit-Change-Number: 11295
Gerrit-PatchSet: 3
Gerrit-Owner: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>

[kudu-CR](gh-pages) Add archived releases section

Posted by "Attila Bukor (Code Review)" <ge...@cloudera.org>.
Hello Mike Percy, Grant Henke, Todd Lipcon, 

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

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

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

Change subject: Add archived releases section
......................................................................

Add archived releases section

Change-Id: I671f58c8c67604876573f31936a5dc764302098b
---
M releases/index.md
1 file changed, 4 insertions(+), 1 deletion(-)


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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I671f58c8c67604876573f31936a5dc764302098b
Gerrit-Change-Number: 11295
Gerrit-PatchSet: 2
Gerrit-Owner: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>

[kudu-CR](gh-pages) Add archived releases section

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

Change subject: Add archived releases section
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11295/3/releases/index.md
File releases/index.md:

http://gerrit.cloudera.org:8080/#/c/11295/3/releases/index.md@19
PS3, Line 19: ### Archived releases
>> To me it means "here's a bunch of old releases that you can still
download if you really need to for some reason, but please don't, use
the current ones instead"

That quote is from the email thread. Given this is your goal, instead maybe we should just add a paragraph with some advice on choosing a version to download? That is lower maintenance overall, and requires less judgment about what is or isn't "archived".

Something advising that there is a lot of active development in Kudu and using the latest version is highly recommended. You could say that we fix a lot of bugs and the stability/usability is better in newer versions. Additionally the new features are nice and getting help is easier if you are on a newer version.



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I671f58c8c67604876573f31936a5dc764302098b
Gerrit-Change-Number: 11295
Gerrit-PatchSet: 3
Gerrit-Owner: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Comment-Date: Mon, 27 Aug 2018 15:38:51 +0000
Gerrit-HasComments: Yes