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

Vinoth Chandar commented on HUDI-1035:
--------------------------------------

these are pojos used in Spark serialization.. if they are unused, should be 
safe to remove.. i.e wont be cases where its instantiated by reflection etc

> Remove unused class KeyLookupResult
> -----------------------------------
>
>                 Key: HUDI-1035
>                 URL: https://issues.apache.org/jira/browse/HUDI-1035
>             Project: Apache Hudi
>          Issue Type: Improvement
>          Components: Code Cleanup
>            Reporter: wangxianghu
>            Assignee: wangxianghu
>            Priority: Minor
>              Labels: pull-request-available
>
> when refactoring hudi-client module, I found there are two KeyLookupResult 
> classes, which are almost the same:
> No.1 -  org.apache.hudi.index.bloom.KeyLookupResult  *(not used anywhere)*
> No.2 - org.apache.hudi.io.HoodieKeyLookupHandle.KeyLookupResult  *(using 
> currently)*
> maybe we should delete the No.1, or replace No.2 with No.1



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to