DaZuiZui commented on PR #18006: URL: https://github.com/apache/iotdb/pull/18006#issuecomment-4901914148
Follow-up update after the sampling-interval clarification: - Removed the runtime validation that required evenly spaced timestamps when `SAMPLE_INTERVAL` is omitted. - Removed the runtime validation that required input gaps to match an explicitly supplied `SAMPLE_INTERVAL`. - When `SAMPLE_INTERVAL` is omitted, FFT now infers the interval from the partition time range as `(last_time - first_time) / (row_count - 1)`. - When `SAMPLE_INTERVAL` is supplied, FFT uses that value directly. Input gaps are not validated against it. - Timestamps are still required to be strictly ascending, and numeric transform values still cannot be null. Verification: ```bash ./mvnw spotless:apply -pl iotdb-core/node-commons,integration-test -P with-integration-tests ./mvnw test -pl iotdb-core/node-commons -am -Dtest=FFTTableFunctionTest -DfailIfNoTests=false -Dsurefire.failIfNoSpecifiedTests=false -DskipITs ./mvnw verify -DskipUTs -Dit.test=IoTDBFFTTableFunctionIT -DfailIfNoTests=false -Dfailsafe.failIfNoSpecifiedTests=false -pl integration-test -am -PTableSimpleIT -P with-integration-tests ``` -- 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]
