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

Jim Apple commented on IMPALA-7556:
-----------------------------------

[~boroknagyz], could you add some details to the Description so someone else 
could pick up this work if you have other obligations? In what way do you want 
to clean u the mess, specifically? Should there be multiple child classes, one 
for each type of file? What counter updates should be moved elsewhere?

> Clean up ScanRange
> ------------------
>
>                 Key: IMPALA-7556
>                 URL: https://issues.apache.org/jira/browse/IMPALA-7556
>             Project: IMPALA
>          Issue Type: Improvement
>            Reporter: Zoltán Borók-Nagy
>            Assignee: Zoltán Borók-Nagy
>            Priority: Major
>
> For IMPALA-7543 I want to add some additional functionality to scan ranges.
> However, the code of the ScanRange class is already quite messy. It handles 
> different types of files, does some buffer management, updates all kinds of 
> counters.
> So, instead of complicating the code further, let's refactor the ScanRange 
> class a bit.



--
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