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

Drop edatachain file support and logic around it #749

Open
shcheklein opened this issue Dec 25, 2024 · 1 comment
Open

Drop edatachain file support and logic around it #749

shcheklein opened this issue Dec 25, 2024 · 1 comment

Comments

@shcheklein
Copy link
Member

We can export / import to / from parquet or json files.

What is the reason for cp command to support edachain as a format?

It makes code more complicated, requires additional care, etc.

I suggest we drop it, unless we have a strong reason not to.

cc @dmpetrov @ilongin

@dmpetrov
Copy link
Member

It would be great to have these files for codification - committing human readable files to Git.

For now we can use .parquet for transferring datasets. But in general, we would need a special extension to distinguish dataset files from the others. However, the extension needs to be renamed to something like .dc and .dcp (binary parquet version).

For now I'd not prioritize this.

What is the reason for cp command to support edachain as a format?

That's not clear - cp should not use this.

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

No branches or pull requests

2 participants