Skip to content
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

related project #56

Open
pickfire opened this issue Nov 11, 2020 · 4 comments
Open

related project #56

pickfire opened this issue Nov 11, 2020 · 4 comments

Comments

@pickfire
Copy link

https://github.com/pickfire/babelfish

This one is missing yaml, but I will add link to this project later.

@bruceadams
Copy link
Owner

babelfish looks nice, @pickfire ! I see that babelfish uses serde_transcode::transcode which looks great. When I was writing yj, I found that using serde_transcode::transcode was not straightforward, due to how serde_yaml is implemented. I don't think that has changed (and I haven't taken on the project of trying to enhance serde_yaml).

@pickfire
Copy link
Author

It's the same for me. That's why there is an open issue pickfire/babelfish#1. Last time I tried writing mini-yaml but I gave up.

@dtolnay
Copy link

dtolnay commented Jun 25, 2022

due to how serde_yaml is implemented

I assume you are referring to prior to dtolnay/serde-yaml#44 being implemented.

Anyway I don't expect that you would observe any performance improvement from using serde_transcode in this tool.

(I am using yj for processing the Google Cloud command line's output. Thanks! gsutil ls -L gs://$bucket | yj | jq -r 'to_entries[] | (.key | split("/") | last) + "," + (.value | objects["Hash (crc32c)"])')

@bruceadams
Copy link
Owner

Thanks @dtolnay ! I had not noticed that enhancement to serde-yaml. Very cool to hear that you are using yj ❤️‼️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants