Github user KurtYoung commented on the issue:

    https://github.com/apache/flink/pull/2810
  
    Hi @tonycox , i want to appologize first for opening another "project 
pushdown" issue when this one is still in active. We actually come up with some 
whole new idea and designs compare to the original design here, so we think 
maybe it will be easier to open another pull request and let's discuss with 
some codes. 
    
    But i noticed that this PR is becoming more similar with our design now 
(only some slightly differences), even some core codes are copied from our pull 
request. I think we should work out a plan to merge our work together. 
    
    A litter opinion here: I think it's pointless to let CsvTableSource be a 
ProjecableTableSource since it simply cannot avoid reading extra fields in the 
first place. 



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to