[GitHub] [drill] dzamo edited a comment on pull request #2282: DRILL-7978: Fixed Width Format Plugin

[GitBox <gi...@apache.org>]

dzamo edited a comment on pull request #2282:
URL: https://github.com/apache/drill/pull/2282#issuecomment-962939201


   @MFoss19 @estherbuchwalter following some [recent chat](https://github.com/apache/drill/pull/2359#issuecomment-962673076) with @paul-rogers and my last comment here, how about a reduced format config such as the following?  The goal is to get to something terse and consistent with what we do for other text formats.
   
   ```json
   "fixedwidth": {
     "type": "fixedwidth",
     "extensions": [
       "fwf"
     ],
     "extractHeader": true,
     "trimStrings": true,
     "columnOffsets": [1, 11, 21, 31],
     "columnWidths": [10, 10, 10, 10]
   }
   ```
   
   Column names and types can already come from a provided schema or aliasing after calls to `CAST()`.  Incidentally, the settings above can be overriden per query using a provided schema too.
   
   There's also a part of that wonders whether we could have justified adding our fixed width functionality to the existing delimited text format reader.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org