Fix Java client time column access throwing ArrayIndexOutOfBoundsException (#17407)

Add tsBlockColumnIndex < 0 guards to all ByTsBlockColumnIndex getter
methods that were missing them. When the time pseudo-column is accessed
through a typed getter other than getLong or getString, throw
StatementExecutionException instead of causing ArrayIndexOutOfBoundsException
from TsBlock.getColumn(-1).

Signed-off-by: Zihan Dai <1436286758@qq.com>

Author: PDGGK

iotdb-client/service-rpc/src/main/java/org/apache/iotdb/rpc/IoTDBRpcDataSet.java

@@ -328,9 +328,16 @@ public boolean getBoolean(String columnName) throws StatementExecutionException
     return getBooleanByTsBlockColumnIndex(getTsBlockColumnIndexForColumnName(columnName));
   }
 
+  // Note: tsBlockColumnIndex < 0 indicates the time pseudo-column in tree model.
+  // Only getLong and getString support reading the time column directly.
+  // All other typed getters throw StatementExecutionException to prevent
+  // ArrayIndexOutOfBoundsException from TsBlock.getColumn with a negative index.
   private boolean getBooleanByTsBlockColumnIndex(int tsBlockColumnIndex)
       throws StatementExecutionException {
     checkRecord();
+    if (tsBlockColumnIndex < 0) {
+      throw new StatementExecutionException("Cannot read boolean from time column");
+    }
     if (!isNull(tsBlockColumnIndex, tsBlockIndex)) {
       lastReadWasNull = false;
       return curTsBlock.getColumn(tsBlockColumnIndex).getBoolean(tsBlockIndex);
@@ -351,6 +358,9 @@ public double getDouble(String columnName) throws StatementExecutionException {
   private double getDoubleByTsBlockColumnIndex(int tsBlockColumnIndex)
       throws StatementExecutionException {
     checkRecord();
+    if (tsBlockColumnIndex < 0) {
+      throw new StatementExecutionException("Cannot read double from time column");
+    }
     if (!isNull(tsBlockColumnIndex, tsBlockIndex)) {
       lastReadWasNull = false;
       return curTsBlock.getColumn(tsBlockColumnIndex).getDouble(tsBlockIndex);
@@ -371,6 +381,9 @@ public float getFloat(String columnName) throws StatementExecutionException {
   private float getFloatByTsBlockColumnIndex(int tsBlockColumnIndex)
       throws StatementExecutionException {
     checkRecord();
+    if (tsBlockColumnIndex < 0) {
+      throw new StatementExecutionException("Cannot read float from time column");
+    }
     if (!isNull(tsBlockColumnIndex, tsBlockIndex)) {
       lastReadWasNull = false;
       return curTsBlock.getColumn(tsBlockColumnIndex).getFloat(tsBlockIndex);
@@ -391,6 +404,9 @@ public int getInt(String columnName) throws StatementExecutionException {
   private int getIntByTsBlockColumnIndex(int tsBlockColumnIndex)
       throws StatementExecutionException {
     checkRecord();
+    if (tsBlockColumnIndex < 0) {
+      throw new StatementExecutionException("Cannot read int32 from time column");
+    }
     if (!isNull(tsBlockColumnIndex, tsBlockIndex)) {
       lastReadWasNull = false;
       TSDataType type = curTsBlock.getColumn(tsBlockColumnIndex).getDataType();
@@ -449,6 +465,9 @@ public Binary getBinary(String columnName) throws StatementExecutionException {
   private Binary getBinaryTsBlockColumnIndex(int tsBlockColumnIndex)
       throws StatementExecutionException {
     checkRecord();
+    if (tsBlockColumnIndex < 0) {
+      throw new StatementExecutionException("Cannot read binary from time column");
+    }
     if (!isNull(tsBlockColumnIndex, tsBlockIndex)) {
       lastReadWasNull = false;
       return curTsBlock.getColumn(tsBlockColumnIndex).getBinary(tsBlockIndex);
@@ -469,6 +488,9 @@ public Object getObject(String columnName) throws StatementExecutionException {
   private Object getObjectByTsBlockIndex(int tsBlockColumnIndex)
       throws StatementExecutionException {
     checkRecord();
+    if (tsBlockColumnIndex < 0) {
+      throw new StatementExecutionException("Cannot read object from time column");
+    }
     if (isNull(tsBlockColumnIndex, tsBlockIndex)) {
       lastReadWasNull = true;
       return null;
@@ -483,10 +505,7 @@ private Object getObjectByTsBlockIndex(int tsBlockColumnIndex)
       case DOUBLE:
         return curTsBlock.getColumn(tsBlockColumnIndex).getObject(tsBlockIndex);
       case TIMESTAMP:
-        long timestamp =
-            (tsBlockColumnIndex == -1
-                ? curTsBlock.getTimeByIndex(tsBlockIndex)
-                : curTsBlock.getColumn(tsBlockColumnIndex).getLong(tsBlockIndex));
+        long timestamp = curTsBlock.getColumn(tsBlockColumnIndex).getLong(tsBlockIndex);
         return convertToTimestamp(timestamp, timeFactor);
       case TEXT:
       case STRING: