agatedb icon indicating copy to clipboard operation
agatedb copied to clipboard

Feature: Ingest External SST

Open wangnengjie opened this issue 2 years ago • 8 comments

Signed-off-by: wangnengjie [email protected]

See details in: https://github.com/tikv/agatedb/issues/187

wangnengjie avatar Aug 23 '22 06:08 wangnengjie

The next_txn_ts won't update when reopening db? @GanZiheng

wangnengjie avatar Aug 24 '22 06:08 wangnengjie

The next_txn_ts won't update when reopening db? @GanZiheng

Yes... I remember it is set to 1.

GanZiheng avatar Aug 24 '22 09:08 GanZiheng

Yes... I remember it is set to 1.

Well, that's... amazing... I debug a while for my failed test. I think I should make another pr to make it works right.

wangnengjie avatar Aug 24 '22 09:08 wangnengjie

Generally done. You can have a look. @GanZiheng

BTW, the rustfmt.toml use many flags that are only available in nightly channel while CI use stable channel. So it just failed though I run make format under nightly channel.

wangnengjie avatar Sep 01 '22 07:09 wangnengjie

Generally done. You can have a look. @GanZiheng

BTW, the rustfmt.toml use many flags that are only available in nightly channel while CI use stable channel. So it just failed though I run make format under nightly channel.

We currently use stable channel for the sake of avoiding CI failing too often. I think there is a format error in this version of code.

GanZiheng avatar Sep 06 '22 16:09 GanZiheng

Generally done. You can have a look. @GanZiheng BTW, the rustfmt.toml use many flags that are only available in nightly channel while CI use stable channel. So it just failed though I run make format under nightly channel.

We currently use stable channel for the sake of avoiding CI failing too often. I think there is a format error in this version of code.

As what I have said, rustfmt.toml use many flags that are only available in nightly channel which makes the result of format different with stable channel. I will use stable channel to format the code later. So why not just remove all these flags that only available in nightly channel?

Does CI often failed in nightly channel? I saw that all tests (except the first one) run in nightly channel for the support of sanitizer...

wangnengjie avatar Sep 06 '22 16:09 wangnengjie

Does CI often failed in nightly channel? I saw that all tests (except the first one) run in nightly channel for the support of sanitizer...

Specifically, failed due to the clippy check. We can consider remove these flags or just turn back to nightly channel, I think that's not matter.

GanZiheng avatar Sep 06 '22 16:09 GanZiheng

Codecov Report

Merging #188 (f241a88) into master (070ff80) will increase coverage by 1.16%. The diff coverage is 94.85%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
+ Coverage   89.63%   90.80%   +1.16%     
==========================================
  Files          39       40       +1     
  Lines        8838     9679     +841     
==========================================
+ Hits         7922     8789     +867     
+ Misses        916      890      -26     

codecov[bot] avatar Sep 07 '22 01:09 codecov[bot]