[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

[Impala Public Jenkins (Code Review)]

Impala Public Jenkins has submitted this change and it was merged.

Change subject: [DOCS] Release note updates for Impala 2.8
..


[DOCS] Release note updates for Impala 2.8

First cut at 'new features' topic.

Includes the Incompatible Changes subtopic for Impala 2.8.

Also did some cleanup throughout the Incompatible
Changes page:

- Took out references to Cloudera release numbers
  from titles.
- Suppressed the display of ancient subtopics from
  the Impala beta days, which are intertwined with
  things like what version of Cloudera Manager was
  supported.

Patch set 3:

More on MT_DOP for COMPUTE STATS.
Address comments from Greg and MJ.

Added incompatible change for IMPALA-4160 - Llama removal.

Also pulled out hidden discussion and obsolete details of no-op
query options.

Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
Reviewed-on: http://gerrit.cloudera.org:8080/5668
Reviewed-by: Jim Apple 
Tested-by: Impala Public Jenkins
---
M docs/shared/impala_common.xml
M docs/topics/impala_incompatible_changes.xml
M docs/topics/impala_new_features.xml
M docs/topics/impala_reservation_request_timeout.xml
M docs/topics/impala_resource_management.xml
M docs/topics/impala_v_cpu_cores.xml
6 files changed, 523 insertions(+), 795 deletions(-)

Approvals:
  Impala Public Jenkins: Verified
  Jim Apple: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
Gerrit-PatchSet: 10
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Ambreen Kazi 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Dimitris Tsirogiannis 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Matthew Jacobs 
Gerrit-Reviewer: Silvius Rus 

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

[Impala Public Jenkins (Code Review)]

Impala Public Jenkins has posted comments on this change.

Change subject: [DOCS] Release note updates for Impala 2.8
..


Patch Set 9: Verified+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
Gerrit-PatchSet: 9
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Ambreen Kazi 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Dimitris Tsirogiannis 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Matthew Jacobs 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

[Impala Public Jenkins (Code Review)]

Impala Public Jenkins has posted comments on this change.

Change subject: [DOCS] Release note updates for Impala 2.8
..


Patch Set 9:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
Gerrit-PatchSet: 9
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Ambreen Kazi 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Dimitris Tsirogiannis 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Matthew Jacobs 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

[Jim Apple (Code Review)]

Jim Apple has posted comments on this change.

Change subject: [DOCS] Release note updates for Impala 2.8
..


Patch Set 9: Code-Review+2

As the RM for 2.8, I'm +2ing this. We can fix nits in follow-ups, but this will 
block 2.9 release notes work otherwise

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
Gerrit-PatchSet: 9
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Ambreen Kazi 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Dimitris Tsirogiannis 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Matthew Jacobs 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

[Jim Apple (Code Review)]

Jim Apple has posted comments on this change.

Change subject: [DOCS] Release note updates for Impala 2.8
..


Patch Set 9:

2.8.0 shipped a month ago. The delay on this patch is going to start preventing 
developers from updating impala_new_features.xml with their changes that they 
think will go in 2.9.0, thus preventing an incremental update method that would 
ensure that file stays up-to-date and can be released along with the actual 
release.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
Gerrit-PatchSet: 9
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Ambreen Kazi 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Dimitris Tsirogiannis 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Matthew Jacobs 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: No

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

[Laurel Hale (Code Review)]

Laurel Hale has posted comments on this change.

Change subject: [DOCS] Release note updates for Impala 2.8
..


Patch Set 9:

(3 comments)

Some nits you can think about.

http://gerrit.cloudera.org:8080/#/c/5668/9/docs/topics/impala_new_features.xml
File docs/topics/impala_new_features.xml:

PS9, Line 225: a flexible way to ingest, modify, or both data within a Kudu 
table.
This phrase doesn't make sense to me because it tries to use "both" as a verb. 
Instead try, "a flexible way to ingest, modify, or perform both on data within 
a Kudu table."


PS9, Line 330: 
 : 
 :   
 : Impala can take advantage of the S3 encrypted 
credential
 : store, to avoid exposing the secret key when 
accessing
 : data stored on S3.
 :   
 : 
 :   
Not sure why this is a sub-set of the uber list. Doesn't look right in the 
build. I think this list item content should be added right after the colon 
that follows the bolded "Security" heading.


PS9, Line 382: Impala debug web UI:
Should this be bolded to call it out as a subheading like you did with 
"Security"?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
Gerrit-PatchSet: 9
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Ambreen Kazi 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Dimitris Tsirogiannis 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Matthew Jacobs 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: Yes

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

[Jim Apple (Code Review)]

Jim Apple has posted comments on this change.

Change subject: [DOCS] Release note updates for Impala 2.8
..


Patch Set 6:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/5668/6/docs/topics/impala_incompatible_changes.xml
File docs/topics/impala_incompatible_changes.xml:

Line 71: and query options (such as V_CPU_CORES) 
remain but do not have any effect. 
> I am occasionally held up for some time resolving a merge conflict or other
To handle that, you can write your replies, but not send them at first. 
Instead, click "Save" in each little yellow comment box. Instead, push your new 
patch set, then, when that works, send the replies.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
Gerrit-PatchSet: 6
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Ambreen Kazi 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Dimitris Tsirogiannis 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Matthew Jacobs 
Gerrit-Reviewer: Silvius Rus 
Gerrit-HasComments: Yes

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

[John Russell (Code Review)]

John Russell has uploaded a new patch set (#9).

Change subject: [DOCS] Release note updates for Impala 2.8
..

[DOCS] Release note updates for Impala 2.8

First cut at 'new features' topic.

Includes the Incompatible Changes subtopic for Impala 2.8.

Also did some cleanup throughout the Incompatible
Changes page:

- Took out references to Cloudera release numbers
  from titles.
- Suppressed the display of ancient subtopics from
  the Impala beta days, which are intertwined with
  things like what version of Cloudera Manager was
  supported.

Patch set 3:

More on MT_DOP for COMPUTE STATS.
Address comments from Greg and MJ.

Added incompatible change for IMPALA-4160 - Llama removal.

Also pulled out hidden discussion and obsolete details of no-op
query options.

Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
---
M docs/shared/impala_common.xml
M docs/topics/impala_incompatible_changes.xml
M docs/topics/impala_new_features.xml
M docs/topics/impala_reservation_request_timeout.xml
M docs/topics/impala_resource_management.xml
M docs/topics/impala_v_cpu_cores.xml
6 files changed, 523 insertions(+), 795 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/68/5668/9
-- 
To view, visit http://gerrit.cloudera.org:8080/5668
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7c47f422e509cec6d3eb8aaa82294b584f393aed
Gerrit-PatchSet: 9
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Ambreen Kazi 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Dimitris Tsirogiannis 
Gerrit-Reviewer: Greg Rahn 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Laurel Hale 
Gerrit-Reviewer: Matthew Jacobs 
Gerrit-Reviewer: Silvius Rus