Fixing dateTimeConvert regression

[xiangfu0]

fixing the regression introduced from: #11839

[snleee]

wow, thanks for the quick fix! LGTM

[codecov-commenter]

Codecov Report

Merging #11971 (5ea37e8) into master (b5e9823) will decrease coverage by 0.02%.
Report is 4 commits behind head on master.
The diff coverage is 20.00%.

@@             Coverage Diff              @@
##             master   #11971      +/-   ##
============================================
- Coverage     61.43%   61.41%   -0.02%     
  Complexity      207      207              
============================================
  Files          2385     2385              
  Lines        129149   129150       +1     
  Branches      19994    19994              
============================================
- Hits          79342    79318      -24     
- Misses        44055    44075      +20     
- Partials       5752     5757       +5     

Flag	Coverage Δ
custom-integration1	<0.01% <0.00%> (ø)
integration	<0.01% <0.00%> (ø)
integration1	<0.01% <0.00%> (ø)
integration2	0.00% <0.00%> (ø)
java-11	61.37% <20.00%> (-0.02%)	⬇️
java-21	61.28% <20.00%> (+0.01%)	⬆️
skip-bytebuffers-false	61.41% <20.00%> (+0.01%)	⬆️
skip-bytebuffers-true	27.59% <0.00%> (-33.68%)	⬇️
temurin	61.41% <20.00%> (-0.02%)	⬇️
unittests	61.41% <20.00%> (-0.02%)	⬇️
unittests1	46.64% <20.00%> (-0.05%)	⬇️
unittests2	27.61% <0.00%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...e/pinot/common/function/TransformFunctionType.java	89.15% <100.00%> (ø)
...a/org/apache/pinot/common/utils/PinotDataType.java	81.64% <0.00%> (-0.17%)	⬇️

... and 24 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!