DaZuiZui commented on PR #18006: URL: https://github.com/apache/iotdb/pull/18006#issuecomment-4891398416
已更新 PR 分支，主要改动如下： - 根据评审意见，将 FFT 计算路径按输入类型拆分：`FLOAT` 输入使用 `FloatFFT_1D`，`DOUBLE` 及其他数值输入使用 `DoubleFFT_1D`，避免 `FLOAT` 输入先转换为 `double` 后统一处理。 - 移除了包内 `DoubleFft1d` 实现，`FFTTableFunction` 改为直接调用 JTransforms 的 `DoubleFFT_1D` / `FloatFFT_1D`。 - 为 `iotdb-core/node-commons` 补充 JTransforms Maven 依赖，并同步更新 `LICENSE` / `LICENSE-binary` 中相关许可证说明。 - 补充并收紧 `FLOAT` 输入单测：mock `Record` 中 `FLOAT` 只能通过 `getFloat()` 读取，确保测试可以覆盖 `FloatFFT_1D` 路径。 - 将相关集成测试中的 FFT 结果断言调整为带 delta 的数值比较，适配 float FFT 计算精度差异。 验证： - `./mvnw spotless:apply -pl iotdb-core/node-commons` - `./mvnw test -pl iotdb-core/node-commons -am -Dtest=FFTTableFunctionTest -DfailIfNoTests=false -Dsurefire.failIfNoSpecifiedTests=false -DskipITs` - `./mvnw dependency:analyze-only -pl iotdb-core/node-commons -DskipTests -DskipITs` - `./mvnw -N apache-rat:check` - `./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]
