search

kite-sdk / kite

Kite SDK
http://kitesdk.org/docs/current/
Apache License 2.0
394 stars 263 forks source link

KITE-1013: Configurable Sample-Based Type Inferencing for JSON and CSV #406

Closed bdrillard closed 7 years ago

bdrillard commented 9 years ago

These commits provide a patch for KITE-1013, to add configurable, sample-based type inferencing for JSON and CSV. It adds the command-line flags -n, --samples which lets the user specify the number of rows or objects to sample when inferencing types for schemas.

KITE-1013: adding command-line param to csv-schema KITE-1013: requiring at least one sample row KITE-1013: adding samples option to JSON-schema KITE-1013: testing CSVSchema on too few samples KITE-1013: removing debug log, fixing test typo KITE-1013: squashing commit history

rdblue commented 9 years ago

Thanks for taking the time to contribute, @bdrillard!

rdblue commented 9 years ago

Nice work! Just a few things to fix and we'll get this merged. Thanks!