Skip to content

Load: support load tsfile with data in partition whose id is negative#12307

Merged
SteveYurongSu merged 6 commits intomasterfrom
load--1-time-partition
Apr 9, 2024
Merged

Load: support load tsfile with data in partition whose id is negative#12307
SteveYurongSu merged 6 commits intomasterfrom
load--1-time-partition

Conversation

@SteveYurongSu
Copy link
Copy Markdown
Member

As title.

@SteveYurongSu SteveYurongSu self-assigned this Apr 9, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2024

Codecov Report

Attention: Patch coverage is 14.28571% with 30 lines in your changes are missing coverage. Please review.

Project coverage is 46.54%. Comparing base (08b3772) to head (5f123b2).
Report is 2 commits behind head on master.

Files Patch % Lines
...e/dataregion/tsfile/timeindex/DeviceTimeIndex.java 0.00% 21 Missing ⚠️
...ine/dataregion/tsfile/timeindex/FileTimeIndex.java 0.00% 5 Missing ⚠️
...otdb/db/exception/PartitionViolationException.java 0.00% 3 Missing ⚠️
...b/db/storageengine/dataregion/tsfile/TsFileID.java 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master   #12307   +/-   ##
=========================================
  Coverage     46.53%   46.54%           
  Complexity       69       69           
=========================================
  Files          3364     3364           
  Lines        206869   206936   +67     
  Branches      24909    24921   +12     
=========================================
+ Hits          96268    96315   +47     
- Misses       110601   110621   +20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SteveYurongSu SteveYurongSu changed the title Load: support load tsfile with data in partition whose id = -1 Load: support load tsfile with data in partition whose id is negative Apr 9, 2024
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 9, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

Copy link
Copy Markdown
Contributor

@yschengzi yschengzi left a comment

Choose a reason for hiding this comment

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

LGTM

@SteveYurongSu SteveYurongSu merged commit cbc7ba5 into master Apr 9, 2024
@JackieTien97 JackieTien97 deleted the load--1-time-partition branch April 9, 2024 15:48
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