[ 
https://issues.apache.org/jira/browse/IMPALA-7839?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16697470#comment-16697470
 ] 

ASF subversion and git services commented on IMPALA-7839:
---------------------------------------------------------

Commit 8c59f0dcef11389ebfde1788aa3237b7b31ab3a4 in impala's branch 
refs/heads/master from [~fredyw]
[ https://git-wip-us.apache.org/repos/asf?p=impala.git;h=8c59f0d ]

IMPALA-7839: Remove code duplication for getting a unique catalog object name

This patch removes code duplication for getting a catalog object
unique name by reusing the code from Catalog::toCatalogObjectKey(
TCatalogObject).

Testing:
- Ran all FE tests

Change-Id: I8218e57210c06fe6f2578a792d518e4b0287f15f
Reviewed-on: http://gerrit.cloudera.org:8080/11928
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>


> Refactor Catalog::toCatalogObjectKey and CatalogObject::getUniqueName to 
> reduce code repetition
> -----------------------------------------------------------------------------------------------
>
>                 Key: IMPALA-7839
>                 URL: https://issues.apache.org/jira/browse/IMPALA-7839
>             Project: IMPALA
>          Issue Type: Improvement
>          Components: Catalog
>            Reporter: Fredy Wijaya
>            Assignee: Fredy Wijaya
>            Priority: Minor
>
> Some examples of code repetition.
> https://github.com/apache/impala/blob/7299a6aedbbb7563e465dcfc6a20a981e9f1b804/fe/src/main/java/org/apache/impala/catalog/Catalog.java#L544-L582
> https://github.com/apache/impala/blob/7299a6aedbbb7563e465dcfc6a20a981e9f1b804/fe/src/main/java/org/apache/impala/catalog/PrincipalPrivilege.java#L146-L150
> https://github.com/apache/impala/blob/7299a6aedbbb7563e465dcfc6a20a981e9f1b804/fe/src/main/java/org/apache/impala/catalog/Table.java#L457-L458



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to