You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Lars Volker (Code Review)" <ge...@cloudera.org> on 2016/04/13 13:26:48 UTC

[Impala-CR](cdh5-trunk) IMPALA-1920: Fix userinfo and authority parsing.

Lars Volker has uploaded a new patch set (#2).

Change subject: IMPALA-1920: Fix userinfo and authority parsing.
......................................................................

IMPALA-1920: Fix userinfo and authority parsing.

IMPALA-1170 fixed parsing of host parts in urls under the presence of
multiple '@' characters. However the parser contained another issue
where userinfo was erroneously extracted if the query part contained
an '@' but no userinfo was present. This change makes the parser only
extract userinfo before the first '/'.

This change also fixes an issue where the authority part was not
correctly parsed in absence of a terminating slash.

Change-Id: I6486aecc7344f40d6f85d70960866fbb45bea2e1
---
M be/src/exprs/expr-test.cc
M be/src/util/url-parser.cc
2 files changed, 33 insertions(+), 23 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6486aecc7344f40d6f85d70960866fbb45bea2e1
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Lars Volker <lv...@cloudera.com>
Gerrit-Reviewer: Alex Behm <al...@cloudera.com>