[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change.

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..


Patch Set 2: Verified+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: No


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged.

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..


[DOCS] Construct "fixed issues" topic for 2.8 release notes

Primarily a link to the relevant JIRA report,
sorted in descending order of priority.

Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Reviewed-on: http://gerrit.cloudera.org:8080/6125
Reviewed-by: John Russell 
Tested-by: Impala Public Jenkins
---
M docs/topics/impala_fixed_issues.xml
1 file changed, 23 insertions(+), 7 deletions(-)

Approvals:
  Impala Public Jenkins: Verified
  John Russell: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Silvius Rus 


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change.

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..


Patch Set 2:

Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/50/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: No


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
John Russell has posted comments on this change.

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..


Patch Set 2: Code-Review+2

+2ing based on Laurel's +1 plus Jim's comment had a straightforward fix.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: No


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
John Russell has posted comments on this change.

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/6125/1/docs/topics/impala_fixed_issues.xml
File docs/topics/impala_fixed_issues.xml:

PS1, Line 49: ec
> Please don't add new CDH references.
Done


PS1, Line 49: bsecti
> Is it possible to stop adding new CDH references? We are using the 
Done. In this case I'll just amend the XML comment. For consistency's sake I'll 
do the cleanup of older similar comments at the same time.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: Yes


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
Hello Laurel Hale,

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

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

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

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..

[DOCS] Construct "fixed issues" topic for 2.8 release notes

Primarily a link to the relevant JIRA report,
sorted in descending order of priority.

Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
---
M docs/topics/impala_fixed_issues.xml
1 file changed, 23 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/25/6125/2
-- 
To view, visit http://gerrit.cloudera.org:8080/6125
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Silvius Rus 


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Laurel Hale (Code Review)
Laurel Hale has posted comments on this change.

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..


Patch Set 1: Code-Review+1

(1 comment)

Did a build and no unexpected errors thrown. Rendered docs look good.

http://gerrit.cloudera.org:8080/#/c/6125/1/docs/topics/impala_fixed_issues.xml
File docs/topics/impala_fixed_issues.xml:

PS1, Line 49: 5.10.x
Is it possible to stop adding new CDH references? We are using the 
/ attributes to switch out as appropriate for 
upstream/downstream rendered docs.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: Yes


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change.

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/6125/1/docs/topics/impala_fixed_issues.xml
File docs/topics/impala_fixed_issues.xml:

PS1, Line 49: 10
Please don't add new CDH references.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: Yes


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
John Russell has uploaded a new change for review.

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

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..

[DOCS] Construct "fixed issues" topic for 2.8 release notes

Primarily a link to the relevant JIRA report,
sorted in descending order of priority.

Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
---
M docs/topics/impala_fixed_issues.xml
1 file changed, 16 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/25/6125/1
-- 
To view, visit http://gerrit.cloudera.org:8080/6125
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7033c61dafb2dbaf4f05a687bed3bd53883a6ff4
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
John Russell has abandoned this change.

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..


Abandoned

Unintended changes to new features list got mixed in. Impractical to resolve 
all conflicts. Going to start over with only the impala_fixed_issues.xml 
changes.

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: I317d32cd732163a22a524df52d5c6d6184f6343d
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Silvius Rus 


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-01-27 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#2).

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..

[DOCS] Construct "fixed issues" topic for 2.8 release notes

Primarily a link to the relevant JIRA report,
sorted in descending order of priority.

Change-Id: I317d32cd732163a22a524df52d5c6d6184f6343d
---
M docs/topics/impala_fixed_issues.xml
M docs/topics/impala_new_features.xml
2 files changed, 58 insertions(+), 22 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/14/5814/2
-- 
To view, visit http://gerrit.cloudera.org:8080/5814
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I317d32cd732163a22a524df52d5c6d6184f6343d
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Silvius Rus 


[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-01-27 Thread John Russell (Code Review)
John Russell has uploaded a new change for review.

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

Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes
..

[DOCS] Construct "fixed issues" topic for 2.8 release notes

Primarily a link to the relevant JIRA report,
sorted in descending order of priority.

Change-Id: I317d32cd732163a22a524df52d5c6d6184f6343d
---
M docs/topics/impala_fixed_issues.xml
1 file changed, 16 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/14/5814/1
-- 
To view, visit http://gerrit.cloudera.org:8080/5814
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I317d32cd732163a22a524df52d5c6d6184f6343d
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell