Skip to content

[IOTDB-5827] Change default multi_dir_strategy to SequenceStrategy and fix original bug#9718

Merged
xingtanzjr merged 4 commits intomasterfrom
multi_dir_change_to_seq
Apr 27, 2023
Merged

[IOTDB-5827] Change default multi_dir_strategy to SequenceStrategy and fix original bug#9718
xingtanzjr merged 4 commits intomasterfrom
multi_dir_change_to_seq

Conversation

@xingtanzjr
Copy link
Contributor

Description

  1. change default multi_dir_strategy to SequenceStrategy
  2. fix original bug in SequenceStrategy where one folder won't used if others space is limited
  3. use diskSpaceWarningThreshold to decide whether a folder is full or not

Test

Tested using 1C1D in fit16 with 1 DataRegion. See the snapshot below. The threshold is set to 0.85 in this test.
5BpUzfYisk

Copy link
Contributor

@HeimingZ HeimingZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. But the codes need running mvn spotless.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@xingtanzjr xingtanzjr merged commit 89b2b96 into master Apr 27, 2023
@xingtanzjr xingtanzjr changed the title Change default multi_dir_strategy to SequenceStrategy and fix original bug [IOTDB-5827] Change default multi_dir_strategy to SequenceStrategy and fix original bug Apr 27, 2023
@HTHou HTHou deleted the multi_dir_change_to_seq branch May 6, 2023 08:20
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