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.

Sign up for GitHub

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

Jump to bottom

VBuffer columns are not saved or laoded correctly #6858

Open
colombod opened this issue Oct 11, 2023 · 4 comments · Fixed by #6860
Open

VBuffer columns are not saved or laoded correctly #6858

colombod opened this issue Oct 11, 2023 · 4 comments · Fixed by #6860
Labels
untriaged New issue has not been triaged

Comments

@colombod
Copy link
Member

When writing a DataFrame to csv file VBuffer columns are saved as ToString() and do not contain the data.
When reading back the csv the column is now a string type describing the size of the data.
@ghost ghost added the untriaged New issue has not been triaged label Oct 11, 2023
@JakeRadMSFT
Copy link
Contributor

@asmirnov82 :)

@asmirnov82 asmirnov82 mentioned this issue Oct 12, 2023
Merged
@ghost ghost added the in-pr label Oct 12, 2023
@asmirnov82
Copy link
Contributor

Saving into csv is fixed by #6860

@ghost ghost removed in-pr untriaged New issue has not been triaged labels Oct 14, 2023
@JakeRadMSFT JakeRadMSFT reopened this Oct 17, 2023
@ghost ghost added the untriaged New issue has not been triaged label Oct 17, 2023
@JakeRadMSFT
Copy link
Contributor

Re-opening to track the loading issue

@asmirnov82
Copy link
Contributor

Loading csv into VBufferDataFrameColumn instead of StringColumn related to #6859

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
untriaged New issue has not been triaged
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix DataFrame Saving csv with VBufferDataFrameColumn
3 participants
@colombod @JakeRadMSFT @asmirnov82