-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support JSON as input (and output?) format #22
Comments
Actually, if we allow JSON as output, the exact output record structure shouldn't be viewed as a huge decision point, since with JSON output users can easily create arbitrary output structures, example: Much bigger question is what should be the output records structure in case of json input and csv output (i.e. query + format conversion) |
Hi @mechatroner, How hard is just doing json.dumps for a very simple object? Then just say that's the limitation for now? Thanks for the good work. Love and peace, |
Hi @JoeCodeswell, Thank you for the feedback! |
Thanks for letting me know, Dmitry.
Love and peace,
Joe
…On Tue, Feb 13, 2024 at 5:57 PM Dmitry Ignatovich ***@***.***> wrote:
Hi @JoeCodeswell <https://github.com/JoeCodeswell>, Thank you for the
feedback!
Yeah, json.dumps will work - I just verified it, but you would need to
add import json in ~/.rbql_init_source.py file. Or you can also use
JSON.stringify in JS version out of the box. It is also better to set the
output format as tsv instead of csv to avoid double quoting.
So output JSON is already supported to some extent.
—
Reply to this email directly, view it on GitHub
<#22 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAYLZJIL52ZCATOY3YPIFMLYTQKYFAVCNFSM4S5HM222U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJUGI4TOMBZGE2Q>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
One of the main problems is supporting JOIN, when we have a query like
select * JOIN other_file.json on a['key'] = b['key']['subkey']
, what should the output records be? One option would be to output [a, b] which would be valid json too. We would also have to create an alternative join map generation that would have to be eval-based, currently we convert join variables to indexes to generate the join map, but with JSON it is not possible.The text was updated successfully, but these errors were encountered: