Skip to content

Fix empty deleteTimeseries requests#18012

Open
Caideyipi wants to merge 1 commit into
apache:masterfrom
Caideyipi:fix-empty-delete-timeseries
Open

Fix empty deleteTimeseries requests#18012
Caideyipi wants to merge 1 commit into
apache:masterfrom
Caideyipi:fix-empty-delete-timeseries

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Summary

  • Treat empty deleteTimeseries path lists as no-op in SessionConnection and SessionPool.
  • Return SUCCESS for empty deleteTimeseries thrift requests after login validation.
  • Add regression coverage for Session and SessionPool empty-list calls.

Fixes #11612

Tests

  • git diff --check
  • mvn "-Ddevelocity.off=true" -pl iotdb-client/session "-Dtest=SessionConnectionTest#testDeleteEmptyTimeseries,SessionPoolTest#testDeleteEmptyTimeseriesList" test

Notes

  • Tried datanode compile with: mvn "-Ddevelocity.off=true" -pl iotdb-core/datanode -DskipTests compile
  • That local compile fails on unrelated existing dependency/generated-source mismatches such as missing IOUtils.readFully, parser symbols, and freemarker aggregation dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sessionPool.deleteTimeseries操作删除空的路径列表时,会报错

1 participant