Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/18274 )

Change subject: IMPALA-10946: Replaced URLDecoder with Hive's unescape
......................................................................

IMPALA-10946: Replaced URLDecoder with Hive's unescape

Hive is using its own implementation of escaping and unescaping
partition pathes, which is similar to URL decoding, but not the same,
for example it does not escape '+'.

Modified Impala to use the same unescaping algorithm to prevent creating
phantom partitions in HMS.

Testing:

The reproduction in the Jira is fixed, "abc " is no longer created in
case "abc+" partition is recovered. "abc\" still works as intended.

Change-Id: Ie821a0795d99eb9aa95131323917c9a3d0e8e1ec
Reviewed-on: http://gerrit.cloudera.org:8080/18274
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java
1 file changed, 19 insertions(+), 27 deletions(-)

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

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie821a0795d99eb9aa95131323917c9a3d0e8e1ec
Gerrit-Change-Number: 18274
Gerrit-PatchSet: 10
Gerrit-Owner: Gergely Fürnstáhl <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>

Reply via email to