PDGGK commented on PR #17408: URL: https://github.com/apache/iotdb/pull/17408#issuecomment-4802405161
Thanks for the suggestion, @HTHou. Separately, I noticed CI was red and it was my change that caused it: making `getObject` reject the time pseudo-column was too aggressive. `getObject` on the time column is a valid, already-tested path (`IoTDBJDBCResultSetTest#testQuery`) and should keep returning the timestamp. I've restored the time-column branch in the `TIMESTAMP` case of `getObjectByTsBlockIndex` and kept the new `tsBlockColumnIndex < 0` guards only on the strictly-typed getters (boolean/int32/float/double/binary); `getLong`, `getString` and `getObject` continue to read the time column. `IoTDBJDBCResultSetTest` passes locally now. -- 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]
