Closed butanian closed 6 years ago
Since we don't do "releases", I feel like it is not a good idea to merge this without implementing the code that use the UploadAsJSON*
fields. It will leave the master in a state where these config options will not work.
@sanjaybv Definitely not going to merge this unless with the accompanying change. Just a separate PR to make it easy to review.
Dropping this PR. If we end up going down the route of supporting JSON only uploads will revisit.
Separate PR for creating new properties, that lets users upload files as JSON instead of CSV if they are not loading into the warehouse and want to query the files directly.