Skip to content

[IOTDB-1087] Flush thread hang in the merge method of LevelCompaction…#2356

Closed
yuqi1129 wants to merge 1 commit intoapache:masterfrom
yuqi1129:IOTDB-1087
Closed

[IOTDB-1087] Flush thread hang in the merge method of LevelCompaction…#2356
yuqi1129 wants to merge 1 commit intoapache:masterfrom
yuqi1129:IOTDB-1087

Conversation

@yuqi1129
Copy link
Contributor

@yuqi1129 yuqi1129 commented Dec 28, 2020

…TsFileManagement

Description

Content1 ...

Content2 ...

Content3 ...


This PR has:

  • been self-reviewed.
    • concurrent read
    • concurrent write
    • concurrent read and write
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods.
  • added or updated version, license, or notice information
  • added comments explaining the "why" and the intent of the code wherever would not be obvious
    for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold
    for code coverage.
  • added integration tests.
  • been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

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

No Coverage information No Coverage information
0.0% 0.0% Duplication

@zhanglingzhe0820
Copy link
Contributor

It seems that you fix this bug by modify lock, but it is not a lock related bug. We are fixing this bug in pr:#2341 . Can you just help us review this pr as it cannot pass windows ci.

@yuqi1129
Copy link
Contributor Author

@zhanglingzhe0820, OK, i will make some test about your PR, thanks for your attention

@zhanglingzhe0820
Copy link
Contributor

as the bug is fixed in pr:#2341, and it is merged, shall we close this pr

@yuqi1129
Copy link
Contributor Author

@zhanglingzhe0820 , OK, i will close it later

@yuqi1129 yuqi1129 closed this Dec 30, 2020
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