Skip to content

Commit

Permalink
Use array_value_to_string in arrow-csv (#3514)
Browse files Browse the repository at this point in the history
* Use array_value_to_string in arrow-csv

* Fix test

* Add datetime_array_value_to_string to allow passing datetime format

* Rollback the tests

* Add option to use RFC3339 in CSV writeR

* Update tests

* Fix linting errors

* fix tests

* Change with_rfc3339 factory method and use Option<&str> instead of &Option<String>

* Keep old tests intact

* Add tests to check rfc3339

* Add back test_conversion_consistency

* Fix clippy errors

* Minor linting issue

* Separate array_value_to_string and datetime_array_value_to_string

* Add back invalid cast test

* Fix linting and clippy errors

* Fix arrow-cast test

* Restructuring

* Fix formatting errors

* Change make_duration_string to use invalid_cast_error

* Fix clippy errors
  • Loading branch information
JayjeetAtGithub authored Feb 2, 2023
1 parent 2b9bbce commit 75a56be
Show file tree
Hide file tree
Showing 4 changed files with 555 additions and 406 deletions.
Loading

0 comments on commit 75a56be

Please sign in to comment.