Github user dongjoon-hyun commented on the issue:

    https://github.com/apache/spark/pull/17980
  
    Thank you for review, @kiszk . 
    - The first one about [adding new ORC 
source](https://github.com/apache/spark/pull/17924) is more smaller than this .
    - Also, there is [more smaller version without vectorized 
part](https://github.com/apache/spark/pull/17943).
    
    If we need to split more, we can split the one adding `Apache ORC` 
dependency into a separate one, too. Is that helpful for review?
    
    In general, about 50% lines are new ORC test codes which is duplicated from 
old ORC test codes. Could you recommend some better way? 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to