MaxGekk opened a new pull request #28582:
URL: https://github.com/apache/spark/pull/28582
### What changes were proposed in this pull request?
Add new methods that accept date-time Java types to the DateFormatter and
TimestampFormatter traits. The methods format input date-time instances to
strings:
- TimestampFormatter:
- `def format(ts: Timestamp): String`
- `def format(instant: Instant): String`
- DateFormatter:
- `def format(date: Date): String`
- `def format(localDate: LocalDate): String`
### Why are the changes needed?
To avoid unnecessary overhead of converting Java date-time types to
micros/days before formatting. Also formatters have to convert input
micros/days back to Java types to pass instances to standard library API.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
By existing tests for toHiveString.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
