Hello Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2777
to look at the new patch set (#4).
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, 34 insertions(+), 23 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/77/2777/4
--
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: 4
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Lars Volker <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
