Copilot commented on code in PR #17085:
URL: https://github.com/apache/iotdb/pull/17085#discussion_r2730238218
##########
integration-test/src/main/java/org/apache/iotdb/it/utils/TsFileTableGenerator.java:
##########
@@ -81,15 +81,28 @@ public void registerTable(
public void generateData(final String tableName, final int number, final
long timeGap)
throws IOException, WriteProcessException {
final List<IMeasurementSchema> schemas =
table2MeasurementSchema.get(tableName);
+ final List<IMeasurementSchema> schemaWithoutTime =
+ schemas.stream()
+ .filter(schema -> !schema.getMeasurementName().equals("time"))
+ .collect(Collectors.toList());
Review Comment:
The filtering logic for TIME columns is inconsistent. The schemas are
filtered by checking if the measurement name equals "time" (line 86), but the
column categories are filtered by checking if the category is TIME (line 102).
If a TIME column has a different name (e.g., "time1"), it won't be filtered
from schemaWithoutTime but will be filtered from the column categories, causing
a size mismatch between the lists passed to the Tablet constructor. Consider
filtering schemas by checking the corresponding category from
columnCategoryList instead of hardcoding the name check.
--
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]
