[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-08-02 Thread Alex Rodoni (Code Review)
Alex Rodoni has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..

IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Reviewed-on: http://gerrit.cloudera.org:8080/13960
Tested-by: Impala Public Jenkins 
Reviewed-by: Bharath Vissapragada 
---
M docs/topics/impala_client.xml
M docs/topics/impala_jdbc.xml
M docs/topics/impala_ports.xml
3 files changed, 185 insertions(+), 39 deletions(-)

Approvals:
  Impala Public Jenkins: Verified
  Bharath Vissapragada: Looks good to me, approved

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 6
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-08-02 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 5: Code-Review+2


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Fri, 02 Aug 2019 20:14:45 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-08-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 5: Verified+1

Build Successful

https://jenkins.impala.io/job/gerrit-docs-auto-test/422/ : Doc tests passed.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 01 Aug 2019 17:58:40 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-08-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 5:

Build Started https://jenkins.impala.io/job/gerrit-docs-auto-test/422/

Testing docs change - this change appears to modify docs/ and no code. This is 
experimental - please report any issues to tarmstr...@cloudera.com or on this 
JIRA: IMPALA-7317


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 01 Aug 2019 17:46:40 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-08-01 Thread Alex Rodoni (Code Review)
Hello Bharath Vissapragada, Michael Ho, Thomas Tauber-Marshall, Tamas Mate, Tim 
Armstrong, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..

IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
---
M docs/topics/impala_client.xml
M docs/topics/impala_jdbc.xml
M docs/topics/impala_ports.xml
3 files changed, 185 insertions(+), 39 deletions(-)


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 5
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-08-01 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/13960/4/docs/topics/impala_client.xml
File docs/topics/impala_client.xml:

http://gerrit.cloudera.org:8080/#/c/13960/4/docs/topics/impala_client.xml@264
PS4, Line 264: -
> nit: typo, the dash is outside the codeph block
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 01 Aug 2019 17:45:49 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-08-01 Thread Tamas Mate (Code Review)
Tamas Mate has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 4: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/13960/4/docs/topics/impala_client.xml
File docs/topics/impala_client.xml:

http://gerrit.cloudera.org:8080/#/c/13960/4/docs/topics/impala_client.xml@264
PS4, Line 264: -
nit: typo, the dash is outside the codeph block



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 01 Aug 2019 10:00:18 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 4: Code-Review+2


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Thu, 01 Aug 2019 01:53:31 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 4: Verified+1

Build Successful

https://jenkins.impala.io/job/gerrit-docs-auto-test/421/ : Doc tests passed.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 22:32:58 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 3: Verified+1

Build Successful

https://jenkins.impala.io/job/gerrit-docs-auto-test/420/ : Doc tests passed.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 22:31:50 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 4:

Build Started https://jenkins.impala.io/job/gerrit-docs-auto-test/421/

Testing docs change - this change appears to modify docs/ and no code. This is 
experimental - please report any issues to tarmstr...@cloudera.com or on this 
JIRA: IMPALA-7317


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 22:21:32 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Alex Rodoni (Code Review)
Hello Bharath Vissapragada, Michael Ho, Thomas Tauber-Marshall, Tamas Mate, Tim 
Armstrong, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..

IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
---
M docs/topics/impala_client.xml
M docs/topics/impala_jdbc.xml
M docs/topics/impala_ports.xml
3 files changed, 185 insertions(+), 39 deletions(-)


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 3:

> Oops, I missed that commit. Thanks for the pointer. I think it is
 > still useful to mention that the server endpoint supports basic and
 > spnego auth.

Done


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 22:14:28 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 3:

Build Started https://jenkins.impala.io/job/gerrit-docs-auto-test/420/

Testing docs change - this change appears to modify docs/ and no code. This is 
experimental - please report any issues to tarmstr...@cloudera.com or on this 
JIRA: IMPALA-7317


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 22:13:52 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Alex Rodoni (Code Review)
Hello Bharath Vissapragada, Michael Ho, Thomas Tauber-Marshall, Tamas Mate, Tim 
Armstrong, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..

IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
---
M docs/topics/impala_client.xml
M docs/topics/impala_jdbc.xml
M docs/topics/impala_ports.xml
3 files changed, 185 insertions(+), 39 deletions(-)


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 2:

Oops, I missed that commit. Thanks for the pointer. I think it is still useful 
to mention that the server endpoint supports basic and spnego auth.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 21:38:56 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 2:

> > > Forgot about this. I think we should talk about authentication
 > > > semantics over the new HTTP protocol. We support HTTP Basic
 > auth.
 > > > SPNEGO is not yet supported.
 > >
 Didn't IMPALA-8783 add the SPNEGo support over HS2 HTTP?
And since the basic and Kerberos SPNEGO are supported over HTTP, just like over 
Binary, I don't think there is anything to call out for HTTP authN. @Bharath?


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 20:40:15 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 2:

> > Forgot about this. I think we should talk about authentication
 > > semantics over the new HTTP protocol. We support HTTP Basic auth.
 > > SPNEGO is not yet supported.
 >
 > Didn't IMPALA-8783 add the SPNEGo support over HS2 HTTP?


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 20:26:07 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 2:

> Forgot about this. I think we should talk about authentication
 > semantics over the new HTTP protocol. We support HTTP Basic auth.
 > SPNEGO is not yet supported.

Didn't IMPALA-8783 added the SPNEGo support over HS2 HTTP?


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 20:02:57 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/13960/2/docs/topics/impala_ports.xml
File docs/topics/impala_ports.xml:

http://gerrit.cloudera.org:8080/#/c/13960/2/docs/topics/impala_ports.xml@401
PS2, Line 401: .
> ..using HiveServer2 protocol.
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 20:01:06 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 2: Code-Review-1

Forgot about this. I think we should talk about authentication semantics over 
the new HTTP protocol. We support HTTP Basic auth. SPNEGO is not yet supported.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 19:53:31 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 2: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/13960/2/docs/topics/impala_ports.xml
File docs/topics/impala_ports.xml:

http://gerrit.cloudera.org:8080/#/c/13960/2/docs/topics/impala_ports.xml@401
PS2, Line 401: .
..using HiveServer2 protocol.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 19:50:39 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 2: Verified+1

Build Successful

https://jenkins.impala.io/job/gerrit-docs-auto-test/418/ : Doc tests passed.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 18:38:30 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 2:

Build Started https://jenkins.impala.io/job/gerrit-docs-auto-test/418/

Testing docs change - this change appears to modify docs/ and no code. This is 
experimental - please report any issues to tarmstr...@cloudera.com or on this 
JIRA: IMPALA-7317


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 18:17:53 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Alex Rodoni (Code Review)
Hello Bharath Vissapragada, Michael Ho, Thomas Tauber-Marshall, Tamas Mate, Tim 
Armstrong, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..

IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
---
M docs/topics/impala_client.xml
M docs/topics/impala_jdbc.xml
M docs/topics/impala_ports.xml
3 files changed, 129 insertions(+), 37 deletions(-)


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/13960/1/docs/topics/impala_client.xml
File docs/topics/impala_client.xml:

http://gerrit.cloudera.org:8080/#/c/13960/1/docs/topics/impala_client.xml@208
PS1, Line 208: HTTP(S)
> Small nit, this is a bit ambiguous for me. I believe it could be something
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 18:16:50 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-31 Thread Tamas Mate (Code Review)
Tamas Mate has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 1:

(1 comment)

Hi Alex,
thank you for the doc.

http://gerrit.cloudera.org:8080/#/c/13960/1/docs/topics/impala_client.xml
File docs/topics/impala_client.xml:

http://gerrit.cloudera.org:8080/#/c/13960/1/docs/topics/impala_client.xml@208
PS1, Line 208: HTTP(S)
Small nit, this is a bit ambiguous for me. I believe it could be something like:

To enable SSL for HiveServer2 HTTP endpoint use --ssl_server_certificate and 
--ssl_private_key.

The 'impala_ssl.xml' could be linked as well for reference.

What do you think?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Tamas Mate 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 16:47:37 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 1: Verified+1

Build Successful

https://jenkins.impala.io/job/gerrit-docs-auto-test/415/ : Doc tests passed.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Bharath Vissapragada 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Ho 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Wed, 31 Jul 2019 00:27:35 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..


Patch Set 1:

Build Started https://jenkins.impala.io/job/gerrit-docs-auto-test/415/

Testing docs change - this change appears to modify docs/ and no code. This is 
experimental - please report any issues to tarmstr...@cloudera.com or on this 
JIRA: IMPALA-7317


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Comment-Date: Wed, 31 Jul 2019 00:06:38 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

2019-07-30 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/13960


Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
..

IMPALA-8456: [DOCS] New HTTP protocol for Impala clients

Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
---
M docs/topics/impala_client.xml
M docs/topics/impala_jdbc.xml
M docs/topics/impala_ports.xml
3 files changed, 127 insertions(+), 37 deletions(-)



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

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