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

ASF GitHub Bot commented on DRILL-951:
--------------------------------------

Github user abhipol commented on a diff in the pull request:

    https://github.com/apache/drill/pull/232#discussion_r43721290
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/TextParsingSettings.java
 ---
    @@ -57,6 +56,14 @@ public void set(TextFormatConfig config){
         this.delimiter = bSafe(config.getFieldDelimiter(), "fieldDelimiter");
         this.comment = bSafe(config.getComment(), "comment");
         this.skipFirstLine = config.isSkipFirstLine();
    +    //TODO: avoid specific extension based check
    --- End diff --
    
    csvh is part of the format plugin config
    added extractHeader property and cleared the TextParsingSettings code. will 
upload with other changes.


> CSV header row should be parsed
> -------------------------------
>
>                 Key: DRILL-951
>                 URL: https://issues.apache.org/jira/browse/DRILL-951
>             Project: Apache Drill
>          Issue Type: New Feature
>          Components: Storage - Text & CSV
>            Reporter: Tomer Shiran
>            Assignee: Abhijit Pol
>             Fix For: Future
>
>
> CSV reader is currently treating header names like regular rows. There should 
> be a way to treat the header row as the column names (optional?).
> I exported this dataset to a CSV: 
> https://data.sfgov.org/Public-Safety/SFPD-Incidents-Previous-Three-Months/tmnf-yvry



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

Reply via email to