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

vinoyang commented on HUDI-1035:
--------------------------------

[~wangxianghu] IMO, we can delete the first one.

> 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