Support for TIME, TIMESTAMPNTZ_NANO, UUID types in Inclusive Metrics Evaluator

[manirajv06]

Added support for following types in VariantExpressionUtil:

PhysicalType.TIME PhysicalType.TIMESTAMPNTZ_NANO PhysicalType.TIMESTAMPTZ_NANO PhysicalType.UUID

In addition to covering unit tests for above types, added tests for TIMESTAMPNTZ and TIMESTAMPTZ types as well.

[manirajv06]

@aihuaxu @RussellSpitzer Can you please review?

[manirajv06]

@rdblue Thank you for setting up the direction on this. Incorporated your feedback. Please check.

[manirajv06]

@aihuaxu @rdblue Can you please take a look?

[manirajv06]

@aihuaxu We are very close to completion. Do you have anymore feedback? If not, Can we request @RussellSpitzer to commit?

[manirajv06]

@RussellSpitzer Can we get this in? Thanks.

[github-actions[bot]]

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

[manirajv06]

@RussellSpitzer Can we merge the changes? @aihuaxu had already approved this.

[aihuaxu]

@huaxingao Can you also take a look at this change?

[huaxingao]

Thanks @manirajv06 for the PR! The change looks good to me.

[huaxingao]

Thanks @manirajv06 for the PR! I will leave the PR open for a couple of more days in case others want to review.

[huaxingao]

Thanks @manirajv06 for the PR! Thanks @aihuaxu @rdblue for the review!