[
https://issues.apache.org/jira/browse/IMPALA-10384?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17248245#comment-17248245
]
ASF subversion and git services commented on IMPALA-10384:
----------------------------------------------------------
Commit eb8b118db5be9a7c8dca9b3229e76c213e19c86c in impala's branch
refs/heads/master from Zoltan Borok-Nagy
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=eb8b118 ]
IMPALA-10384: Make partition names consistent between BE and FE
In the BE we build partition names with the trailing char '/'. In the FE
we build partition names without a trailing char. We should make this
consistent because this causes some annoying string adjustments in
the FE and can cause hidden bugs.
This patch creates partition names without the trailing '/' both in
the BE and the FE. This follows Hive's behavior that also prints
partition names without the trailing '/'.
Testing:
* Ran exhaustive tests
Change-Id: I7e40111e2d1148aeb01ebc985bbb15db7d6a6012
Reviewed-on: http://gerrit.cloudera.org:8080/16850
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
> Make partition names consistent between BE and FE
> -------------------------------------------------
>
> Key: IMPALA-10384
> URL: https://issues.apache.org/jira/browse/IMPALA-10384
> Project: IMPALA
> Issue Type: Bug
> Reporter: Zoltán Borók-Nagy
> Assignee: Zoltán Borók-Nagy
> Priority: Major
>
> In the BE we build partition names with the trailing char '/'. In the FE we
> build partition names without a trailing char. We should make this consistent.
> I think the correct is the one without the trailing '/'. Hive also prints
> partition names without the trailing '/'. Iceberg also expects partition
> names without the trailing '/'.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
