Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IOTDB-2995]SchemaFileSketcher will not delete empty .pst now #5653

Merged
merged 1 commit into from
Apr 25, 2022

Conversation

bigreybear
Copy link
Contributor

Before this PR, SchemaFileSketcher will delete an empty .pst file since it is useless. Then we found it is confusing for users, so now it will leave the empty file as it was.

@qiaojialin qiaojialin merged commit ff41616 into apache:master Apr 25, 2022
xinzhongtianxia pushed a commit to xinzhongtianxia/iotdb that referenced this pull request Apr 27, 2022
* remotes/upstream/master:
  [IOTDB-3005] Increase read and write permissions in ConfigExecution (apache#5664)
  [IOTDB-3007] Separate DataNode and IoTDB (apache#5668)
  [IOTDB-2968] RatisConsensus snapshot implementation (apache#5623)
  [IOTDB-2958]create a simple schemaCache for datanode metadata (apache#5602)
  [IOTDB-2912] MPP: FilterOperator (apache#5645)
  fix the issue that FragmentInstance's status cannot be updated to FINISHED in some scenario (apache#5660)
  Support set storage group in standalone mpp mode (apache#5655)
  Optimize source handle error handling (apache#5659)
  [IOTDB-2999][Metric] remove useless config and fix default value error. (apache#5657)
  [IOTDB-2966]fix CompactionTaskManagerTest CI (apache#5610)
  Fix schema file with cache of size 1 (apache#5651)
  will not delete empty .pst now (apache#5653)

# Conflicts:
#	server/src/main/java/org/apache/iotdb/db/mpp/sql/analyze/Analyzer.java
#	server/src/main/java/org/apache/iotdb/db/mpp/sql/planner/LogicalPlanner.java
#	server/src/main/java/org/apache/iotdb/db/mpp/sql/planner/plan/node/PlanNodeType.java
@bigreybear bigreybear deleted the i2995 branch June 8, 2022 01:47
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.

3 participants