[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 6: Verified+1


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 6
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Tue, 06 Mar 2018 00:47:59 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..

IMPALA-6553: [DOCS] load_catalog_in_background default change

Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Reviewed-on: http://gerrit.cloudera.org:8080/9389
Reviewed-by: John Russell 
Tested-by: Impala Public Jenkins
---
M docs/shared/impala_common.xml
M docs/topics/impala_invalidate_metadata.xml
2 files changed, 38 insertions(+), 10 deletions(-)

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

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 7
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 6:

Build started: https://jenkins.impala.io/job/gerrit-docs-submit/205/


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 6
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Tue, 06 Mar 2018 00:37:57 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-05 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 5:

> Uploaded patch set 6.

Removed extra spaces.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Mon, 05 Mar 2018 21:38:59 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-05 Thread Alex Rodoni (Code Review)
Hello John Russell, Balazs Jeszenszky, Tim Armstrong, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..

IMPALA-6553: [DOCS] load_catalog_in_background default change

Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
---
M docs/shared/impala_common.xml
M docs/topics/impala_invalidate_metadata.xml
2 files changed, 38 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/89/9389/6
--
To view, visit http://gerrit.cloudera.org:8080/9389
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 6
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-05 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 5:

> (4 comments)
 >
 > A couple of comments inline, plus one for another file that I'm not
 > sure how to mark in the source code:
 >
 > In impala_invalidate_metadata.xml, there's another reference to
 > load_catalog_in_background:
 >
 > table. (This checking does not apply if you have set the
 > catalogd configuration option
 > --load_catalog_in_background=false.) Impala
 > reports any lack of write permissions as an
 >
 > Since the user no longer needs to explicitly set a value of false,
 > perhaps soften this wording a bit and say it doesn't apply when the
 > setting is false, which it is by default.

DONE


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Mon, 05 Mar 2018 21:29:59 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-05 Thread Alex Rodoni (Code Review)
Hello John Russell, Balazs Jeszenszky, Tim Armstrong, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..

IMPALA-6553: [DOCS] load_catalog_in_background default change

Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
---
M docs/shared/impala_common.xml
M docs/topics/impala_invalidate_metadata.xml
2 files changed, 38 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/89/9389/5
--
To view, visit http://gerrit.cloudera.org:8080/9389
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-05 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 4:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml@3448
PS4, Line 3448:  id="ul_h2v_pjf_ycb"
> Remove this unneeded id= attribute. There is an Oxygen setting that control
Done


http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml@3450
PS4, Line 3450: will be
> General style tip: avoid future tense where possible. In this case, say the
Done


http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml@3458
PS4, Line 3458: will aim to
> Remove future tense again, plus "aim to" is a little bit colloquial. Either
Done


http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml@3463
PS4, Line 3463:  id="ul_fx4_hlf_ycb"
> Another unnecessary id= to remove. The presence of IDs suggests that these
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Mon, 05 Mar 2018 21:19:10 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-05 Thread John Russell (Code Review)
John Russell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 4:

(4 comments)

A couple of comments inline, plus one for another file that I'm not sure how to 
mark in the source code:

In impala_invalidate_metadata.xml, there's another reference to 
load_catalog_in_background:

  table. (This checking does not apply if you have set the 
catalogd configuration option
  --load_catalog_in_background=false.) Impala reports any 
lack of write permissions as an

Since the user no longer needs to explicitly set a value of false, perhaps 
soften this wording a bit and say it doesn't apply when the setting is false, 
which it is by default.

http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml@3448
PS4, Line 3448:  id="ul_h2v_pjf_ycb"
Remove this unneeded id= attribute. There is an Oxygen setting that controls 
which elements automatically get id= added - please make to remove  and 
 from that setting on your machine.


http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml@3450
PS4, Line 3450: will be
General style tip: avoid future tense where possible. In this case, say the 
metadata _is_ loaded when...


http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml@3458
PS4, Line 3458: will aim to
Remove future tense again, plus "aim to" is a little bit colloquial. Either 
"the catalog service loads metadata..." if it always happens, or "the catalog 
service attemps to load metadata..." if it might or might not happen.


http://gerrit.cloudera.org:8080/#/c/9389/4/docs/shared/impala_common.xml@3463
PS4, Line 3463:  id="ul_fx4_hlf_ycb"
Another unnecessary id= to remove. The presence of IDs suggests that these 
specific lists might be conref'ed someplace else, but actually only the 
enclosing paragraph is conref'ed. Therefore only that paragraph needs an ID. 
(If someone in future wants to remove these lists or change them into some 
different kind of element, it's helpful to know, based on the lack of id= 
attribute, that there are no conrefs pointing to the  and  elements.)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Mon, 05 Mar 2018 20:01:53 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-03-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 4: Code-Review+1


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 01 Mar 2018 19:10:33 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-02-22 Thread Balazs Jeszenszky (Code Review)
Balazs Jeszenszky has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 4: Code-Review+1

LGTM, after looking around commit comments are often omitted for docs changes.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 22 Feb 2018 21:55:24 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-02-22 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/9389/3/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

http://gerrit.cloudera.org:8080/#/c/9389/3/docs/shared/impala_common.xml@3475
PS3, Line 3475: increasing
> potentially increasing
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 22 Feb 2018 18:04:37 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-02-22 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 3:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/9389/3/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

http://gerrit.cloudera.org:8080/#/c/9389/3/docs/shared/impala_common.xml@3465
PS3, Line 3465: Both on startup and after a global or a table-specific metadata
  : invalidate, the metadata load for tables that 
actual queries can
  : get backed up because the catalog service could 
spend time
  : loading metadata for another table. During such 
load, a request
  : for prioritized load by a query will be put on 
hold,
> This should be simpler.
Done


http://gerrit.cloudera.org:8080/#/c/9389/3/docs/shared/impala_common.xml@3470
PS3, Line 3470: random long-running queries that are difficult
  : for you to diagnose.
> remove 'for you'
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 22 Feb 2018 18:04:07 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-02-22 Thread Alex Rodoni (Code Review)
Hello John Russell, Balazs Jeszenszky, Tim Armstrong, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..

IMPALA-6553: [DOCS] load_catalog_in_background default change

Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
---
M docs/shared/impala_common.xml
1 file changed, 33 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/89/9389/4
--
To view, visit http://gerrit.cloudera.org:8080/9389
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-02-22 Thread Balazs Jeszenszky (Code Review)
Balazs Jeszenszky has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..


Patch Set 3:

(3 comments)

Add commit comment?

http://gerrit.cloudera.org:8080/#/c/9389/3/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

http://gerrit.cloudera.org:8080/#/c/9389/3/docs/shared/impala_common.xml@3465
PS3, Line 3465: Both on startup and after a global or a table-specific metadata
  : invalidate, the metadata load for tables that 
actual queries can
  : get backed up because the catalog service could 
spend time
  : loading metadata for another table. During such 
load, a request
  : for prioritized load by a query will be put on 
hold,
This should be simpler.
Consider: 'Background load can interfere with query-specific metadata loading. 
This can happen on startup or after invalidating metadata, with a duration 
depending on the amount of metadata [making it difficult to diag]'


http://gerrit.cloudera.org:8080/#/c/9389/3/docs/shared/impala_common.xml@3470
PS3, Line 3470: random long-running queries that are difficult
  : for you to diagnose.
remove 'for you'


http://gerrit.cloudera.org:8080/#/c/9389/3/docs/shared/impala_common.xml@3475
PS3, Line 3475: increasing
potentially increasing



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Balazs Jeszenszky 
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: John Russell 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 22 Feb 2018 09:57:34 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-02-21 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded a new patch set (#3). ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..

IMPALA-6553: [DOCS] load_catalog_in_background default change

Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
---
M docs/shared/impala_common.xml
1 file changed, 35 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/89/9389/3
-- 
To view, visit http://gerrit.cloudera.org:8080/9389
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-02-21 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded a new patch set (#2). ( 
http://gerrit.cloudera.org:8080/9389 )

Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..

IMPALA-6553: [DOCS] load_catalog_in_background default change

Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
---
M docs/shared/impala_common.xml
1 file changed, 37 insertions(+), 7 deletions(-)


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 


[Impala-ASF-CR] IMPALA-6553: [DOCS] load catalog in background default change

2018-02-21 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/9389


Change subject: IMPALA-6553: [DOCS] load_catalog_in_background default change
..

IMPALA-6553: [DOCS] load_catalog_in_background default change

Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
---
M docs/shared/impala_common.xml
1 file changed, 37 insertions(+), 6 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I548b2d1532c12f8d3c795a940b7f980482ecf09b
Gerrit-Change-Number: 9389
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni