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

Java Pojo attribute to column mapping #125

Open
dhanan-np opened this issue May 24, 2023 · 1 comment
Open

Java Pojo attribute to column mapping #125

dhanan-np opened this issue May 24, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@dhanan-np
Copy link

dhanan-np commented May 24, 2023

It would be great if we could get a feature like OpenCSV where we can directly bind the csv columns to the Attributes of Java Pojo

@dhanan-np dhanan-np added the enhancement New feature or request label May 24, 2023
@devinrsmith devinrsmith self-assigned this May 24, 2023
@devinrsmith devinrsmith added this to the backlog milestone May 24, 2023
@devinrsmith
Copy link
Member

Hi - thanks for the suggestion. For certain use cases, column mapping to a POJO is a great idea. I believe this feature could be built as a wrapper around the existing interfaces. It may be best to do the mapping "on the fly" so the user doesn't pay the cost of the POJO until they read that row.

Given that deephaven-csv parses and presents the results as column-oriented structures, I'm curious what the performance would be to map back into a POJO per row. I'm guessing it would still be faster than row-oriented approaches that other CSV parsers use, but that is something we'd need to benchmark.

If this is something you are interested in providing a PR for, we're happy to provide further guidance!

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

No branches or pull requests

2 participants