HIVE-29568: Vectorized PTF is throwing RuntimeException

[mdayakar]

HIVE-29568: Vectorized PTF is throwing RuntimeException

What changes were proposed in this pull request?

As a part of vectorized PTF execution, while finding the order by columns datatype when constant value is provided then some times like VOID, INTERVAL_DAY_TIME are missing so added them to support Vectorized PTF execution.

Why are the changes needed?

Can not use Vectorized PTF execution windowing function order by NULL or INTERVAL '1' DAY if the changes are not done.

Does this PR introduce any user-facing change?

No

How was this patch tested?

q file tests are added
mvn -Dtest=TestMiniLlapLocalCliDriver -Dqfile=vector_ptf_windowing_over_const.q -pl itests/qtest -Pitests

[thomasrebele]

LGTM, I'm not an expert in vectorization, though.

[kasakrisz]

LGTM

[kasakrisz]

cc @abstractdog

[abstractdog]

thanks for this patch so far @mdayakar ! I believe the q file would be a bit more valuable if it actually contains some rows, even if the original exception was handled
also in this case, a non-vectorized reference query is usually added for correctness sanity-check

[mdayakar]

thanks for this patch so far @mdayakar ! I believe the q file would be a bit more valuable if it actually contains some rows, even if the original exception was handled also in this case, a non-vectorized reference query is usually added for correctness sanity-check

Thanks for the review @abstractdog , @thomasrebele and @kasakrisz
Fixed the comment given by @abstractdog , inserted some data and the results are compared with and without vectorized ptf execution.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud