You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@calcite.apache.org by "Vladimir Sitnikov (JIRA)" <ji...@apache.org> on 2014/11/21 19:08:33 UTC

[jira] [Created] (CALCITE-472) Split CSV adapter to simpler parts for better readability

Vladimir Sitnikov created CALCITE-472:
-----------------------------------------

             Summary: Split CSV adapter to simpler parts for better readability
                 Key: CALCITE-472
                 URL: https://issues.apache.org/jira/browse/CALCITE-472
             Project: Calcite
          Issue Type: Bug
    Affects Versions: 1.0.0-incubating
            Reporter: Vladimir Sitnikov
            Assignee: Julian Hyde


It might be better to split adapter in different classes/packages/modules so each feature is easily observable:
1) "Demonstration of CSV full scan"
2) "Demonstration of CSV table that returns just required columns"
3) "Demonstration of CSV filterable table"
4) "Demonstration of table that returns _MAP"
etc.
Currently lots of features are mixed in a single implementation, thus it is a bit tricky to figure out how CsvSmart table differs from CsvDump table. Earlier I somehow managed to get over it, however that was very confusing.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)