agavra opened a new pull request, #9546: URL: https://github.com/apache/pinot/pull/9546
This PR supports `TIMESTAMP WITH TIME ZONE` type in Pinot. All specifications are intended to follow https://www.postgresql.org/docs/current/datatype-datetime.html The change required the following steps: 1. add support to Calcite because it isn't yet supported (we will likely contribute this back, see https://issues.apache.org/jira/browse/CALCITE-5301) 2. change usage of `Timestamp` to `LocalDateTime` so that we can introduce the zoned counterpart, `OffsetDateTime` 3. propagate the change through the code base 4. introduce `PinotJsonTimeModule` to support JSON SerDe with the semantics that we want 5. See `TimestampUtil` for the custom handling of timestamps5. 6. See `TimestampQueriesTest` for some end-to-end examples of usage Note that the following are still TODO: - end-to-end quickstart tests that also use `TIMESTAMP WITH TIME ZONE` - a few more pointed test cases - conversion between `TIMESTAMP WITH TIME ZONE` and `TIMESTAMP WITHOUT TIME ZONE` will be done as part of #9477 - we will need a follow up to allow registering functions with the same name but different input types in the scalar function registry -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
