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

Web UI CSV Upload followup #1041

Open
2 of 6 tasks
niloc132 opened this issue Aug 11, 2021 · 5 comments
Open
2 of 6 tasks

Web UI CSV Upload followup #1041

niloc132 opened this issue Aug 11, 2021 · 5 comments
Assignees
Labels
Milestone

Comments

@niloc132
Copy link
Member

niloc132 commented Aug 11, 2021

  • Support LocalDate
  • Support LocalTime
  • Support BigInteger
  • Support BigDecimal
  • Support DateTime
  • Support other java primitives?
@niloc132 niloc132 added the jsapi label Aug 11, 2021
@niloc132 niloc132 added this to the Backlog milestone Aug 11, 2021
@niloc132 niloc132 self-assigned this Aug 11, 2021
@niloc132
Copy link
Member Author

DateTime is handled via #1565, as are other java primitives.

niloc132 added a commit that referenced this issue Nov 19, 2021
The JS InputTable.addRow method now shares an implementation with the CSV upload mechanism (using flight DoPut, exposed as IdeConnection.newTable), and both can parse from a variety of formats. DateTime objects are supported now by this as well.

In order to support this, the JS Table now handles attributes sent from the server. More changes are expected here, from #1484 etc.

This patch also fixes a bug where the server wouldn't correctly respond to an InputTableService call.

Fixes #1451
Fixes #1271 
Partial #1041
@hythloda
Copy link
Contributor

hythloda commented Dec 22, 2021

UI upload of CSV file with DateTime values such as :
"2021-12-22T10:41:19.003"
still return an error:
java.lang.IllegalArgumentException: No support for handling data of type datetim e

yes the e is separate which I thought weird.

File does not load. File was created with write_csv and the column values of the table are type DateTime.

File loads fine with read_csv
@niloc132

@niloc132
Copy link
Member Author

BigInteger/BigDecimal solved by #1627

@niloc132
Copy link
Member Author

niloc132 commented Jun 7, 2023

BigInteger/decimal are not yet solved, we can read but not yet write them.

@niloc132
Copy link
Member Author

niloc132 commented Jun 7, 2023

Consider when working on #188

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

No branches or pull requests

2 participants