search

MeltanoLabs / target-athena

Singer.io Target for AWS Athena.
Other
5 stars 16 forks source link

Updated default to jsonl #38

Closed visch closed 2 years ago

visch commented 2 years ago

Closes #27

edgarrmondragon commented 2 years ago

Thank you @visch!

@pnadolny13 wdyt? I see you're already using JSONL, so this wouldn't be a breaking change for you.

cc @aaronsteers

pnadolny13 commented 2 years ago

Thank you @visch!

@pnadolny13 wdyt? I see you're already using JSONL, so this wouldn't be a breaking change for you.

cc @aaronsteers

@visch thanks - this is great!

@edgarrmondragon yes I'm using jsonl partially because of the same issue raised here. I've run it with csv and had misaligned columns but jsonl worked out of the box so I just stuck with that. I didnt have time to look into whether it was my misconfiguration or a bug.