-
Notifications
You must be signed in to change notification settings - Fork 74
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 YAML input #70
Comments
It should be easy to add -- just follow the pattern of the JSON operator. Currently deeply nested structures aren't well supported, neither is accessing them -- you'd need to hack around that for now the same way JSON does, I think. But when I add better support for nested structures YAML will get it too |
My sincere apologies -- I meant a YAML parser. ( YAML output probably comes as a very natural next step, though ) |
After a bit more thought, this may be quite tricky. Currently, things operate on a line-by-line basis. You'd need to somehow flatten your YAML structure onto a single line or significantly refactor that way input works. |
So for |
|
@gf-mse are your yaml documents separated by |
No, but it could be achieved. Worse comes to worse, I can pre-parse it and add a What actually makes me think if I shall open another issue and ask whether there are plans for some plugin mechanism ) |
I doubt I'd add a plugin mechanism -- the CLI is such that it's generally
better to `|` from one program to another to handle more specific task. I'm
considering handling indentation grouping of records, but it adds
considerable complexity to the ingest pipeline which is currently very
simple
…On Sat, Jul 6, 2019 at 11:38 AM Georgy (George) Fedorov < ***@***.***> wrote:
No, but it could be achieved. Worse comes to worse, I can pre-parse it and
add a --- on every de-indentation, or next top-level list/dictionary Item.
What actually makes me think if I shall open another issue and ask whether
there are plans for some plugin mechanism )
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#70?email_source=notifications&email_token=AADYKZZ2ISBXTBGHC6UI6ZDP6DRCJA5CNFSM4H24RQAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZK6M5I#issuecomment-508946037>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADYKZ65IZ2DDVWXJGLUYZLP6DRCJANCNFSM4H24RQAA>
.
|
Just discovered this project ( which is an awesome idea, btw ); I wonder if it supports
| yaml
-- and if not, then how hard it would be to add that ?The text was updated successfully, but these errors were encountered: