Gabriel39 commented on code in PR #68532:
URL: https://github.com/apache/doris/pull/68532#discussion_r4124424310
##########
fe/be-java-extensions/max-compute-connector/src/main/java/org/apache/doris/maxcompute/MaxComputeJniWriter.java:
##########
@@ -565,8 +565,24 @@ private void fillArrowVectorStreaming(VectorSchemaRoot
root, int colIdx, OdpsTyp
vec.setValueCount(numRows);
break;
}
- case DATETIME:
case TIMESTAMP: {
+ // TIMESTAMPTZ's JNI carrier is UTC and must retain
microseconds on write.
+ org.apache.arrow.vector.TimeStampVector vec =
+ (org.apache.arrow.vector.TimeStampVector)
root.getVector(colIdx);
+ vec.allocateNew(numRows);
+ for (int i = 0; i < numRows; i++) {
+ if (vc.isNullAt(rowOffset + i)) {
+ vec.setNull(i);
+ } else {
Review Comment:
Acknowledged as an old-FE/new-BE compatibility issue for MaxCompute legacy
DATETIMEV2 writes. The requested scope explicitly excludes rolling-upgrade and
compatibility fixes, so the old JVM-zone interpretation is intentionally not
restored in this PR. The PR description now records this known limitation.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
