Internal Jenkins has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/2777 Reviewed-by: Lars Volker <[email protected]> Tested-by: Internal Jenkins --- M be/src/exprs/expr-test.cc M be/src/util/url-parser.cc 2 files changed, 34 insertions(+), 23 deletions(-) Approvals: Lars Volker: Looks good to me, approved Internal Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2777 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6486aecc7344f40d6f85d70960866fbb45bea2e1 Gerrit-PatchSet: 6 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: Internal Jenkins Gerrit-Reviewer: Lars Volker <[email protected]>
