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

write_csv fails to write arrays correctly #4037

Open
cpwright opened this issue Jun 21, 2023 · 1 comment
Open

write_csv fails to write arrays correctly #4037

cpwright opened this issue Jun 21, 2023 · 1 comment
Assignees
Labels
bug Something isn't working core Core development tasks csv
Milestone

Comments

@cpwright
Copy link
Contributor

Description

If you have a table with a Java array column, then the write_csv call dumps out a toString representation.

Steps to reproduce

x=empty_table(10).update("Y=new String[0]")
write_csv(x, "/tmp/z.csv")

Expected results

A file containing "[]".

Actual results

[charleswright@charleswright-verm17d ~]$ cat /tmp/z.csv
Y
[Ljava.lang.String;@5897009e
[Ljava.lang.String;@71cbb466
[Ljava.lang.String;@57851ad6
....

Versions

  • Deephaven: 0.25.2
@cpwright cpwright added bug Something isn't working triage labels Jun 21, 2023
@rcaudy rcaudy self-assigned this Jun 21, 2023
@rcaudy rcaudy added core Core development tasks csv and removed triage labels Jun 21, 2023
@rcaudy rcaudy added this to the Backlog milestone Jun 21, 2023
@rcaudy
Copy link
Member

rcaudy commented Jun 21, 2023

I'm not sure there's a standard way to emit arrays in CSV files. Certainly raw toString() isn't ideal, however. Putting this on the backlog for now, until we do a project to improve CSV writing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working core Core development tasks csv
Projects
None yet
Development

No branches or pull requests

2 participants