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

kvserver: repro L0 snap ingestion #80611

Closed
wants to merge 4 commits into from

Conversation

tbg
Copy link
Member

@tbg tbg commented Apr 27, 2022

Repro for #80589.

100% of all snaps go to L0, see log.txt.

Release note: None

nicktrav and others added 2 commits April 26, 2022 15:00
```
b33d6e17 internal/keyspan: remove bounds from FragmentIterator
07e7075b internal/keyspan: fix SeekLT bug
4c199d34 sstable: add scaffolding to enable sstable parallelism
338611b0 sstable: fix indexBlockBuf.shouldFlush by accounting for inflight data
8decd08c db: pool range-key iterator structs, remove Clone
d8ca1fc6 keyspan: implement Truncate in terms of Filter
5926e724 keyspan: add `keyspan.Filter`
7582b880 db,record: add internal metrics for throughput, latency and utilization
8b3db52b internal/keyspan: Emit empty spans where there are no range keys
4a0889f8 db: add IngestOperationStats for admission control
194770c8 internal/metamorphic/crossversion: add TestMetaCrossVersion
785bb36d internal/keyspan: clean up InterleavingIter
b9bcae65 db: generate delete compaction hints automatically in tests
4e77c077 db: update comment on L0StopWritesThreshold
99d8a549 internal/metamorphic: add support for an initial database state
e232404f vfs: add CloneSync, CloneSkip options for vfs.Clone
7accf3df internal/metamorphic: use a wal_dir within the data directory
886e91f6 internal/keyspan: tweak DefragmentReducer return types
31def7d7 db: add additional deletion only hint test case; update diagram
1793e521 sstable: enforce sstable size estimation invariants
e4927968 db: implement rangedel defragmentation using keyspan.DefragmentingIter
41895ae0 internal/keyspan: reduce over defragmented spans
c55c1d80 db: check ordering invariant for sublevel compactions
322f7e9c internal/keyspan: genercize masking
cca64783 compaction: add L0CompactionFileThreshold compaction trigger
5cd60640 db: respect changed filters in SetOptions
8cc87859 db: add additional table cache test for the Open function
c8e7d414 db: add range key count to table stats
ec4dcaa7 db: factor out some table stats helpers
fb7ccaa2 internal/arenaskl: Overflow fixes
53fd136b internal/keyspan: move and refactor range key iterators as FragmentIterators
c6ff6904 db: use sublevels for l0 manual compactions
8e8ff2be db: add Iterator.SetOptions for modifying iterator options
1b227d2c db: factor out point iterator initialization
3e86958e docs/rfc: flushable ingested sstable addition
0b5f42a0 internal/keyspan: Add LevelIter for range keys
d7fb4eb2 docs/rfc: flushable ingested sstable
92c22536 db: remove unused variable from loop
7ad3551b *: refactor keyspan.Span to represent a set of keys
```

Release note:
Repro for cockroachdb#80589.

100% of all snaps go to L0, see log.txt.

Release note: None
@tbg tbg requested review from a team as code owners April 27, 2022 05:37
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@tbg tbg marked this pull request as draft April 27, 2022 05:37
@tbg tbg removed request for a team April 27, 2022 05:37
@tbg tbg closed this Apr 27, 2022
@tbg tbg reopened this Apr 28, 2022
tbg added 2 commits April 28, 2022 14:56
Looks like the chunky part doesn't go to L0.

```
W220428 13:55:54.088449 603 kv/kvserver/pkg/kv/kvserver/replica_raftstorage.go:952 ⋮ [n3,s3,r45/2:{-}] 65  XXX ingestion into L0: {Bytes:129007245 ApproxIngestedIntoL0Bytes:6311}
W220428 13:55:59.208826 200 kv/kvserver/pkg/kv/kvserver/replica_raftstorage.go:952 ⋮ [n2,s2,r45/3:{-}] 67  XXX ingestion into L0: {Bytes:129007393 ApproxIngestedIntoL0Bytes:6459}
```

Release note: None
@tbg tbg closed this Apr 29, 2022
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