Botond Dénes
Botond Dénes
And: * https://github.com/scylladb/scylladb/pull/11614.
Rebased on master, on top of the now merged split out PRs. Will kick-off BYO once CI approves.
> CI state **`FAILURE`** - https://jenkins.scylladb.com/job/releng/job/Scylla-CI/2526/ Resharding test timed out. Re-kicked CI.
This was dequeued as it was suspected that it causes failures in next promotion.
v4: * Add (and use) `position_in_partition{_view}::for_partition_{start,end}()`. * Raise validation level to token. The raised validation level found further violations, possibly due to https://github.com/scylladb/scylladb/pull/11886, so don't merge yet.
> CI state **`FAILURE`** - https://jenkins.scylladb.com/job/releng/job/Scylla-CI/2328/ There are some tests failing, please look into it.
Where changelog?
> > What is the use-case for these warnings? We recently implemented graceful handling of tombstones via empty pages, so tombstones should not cause timeouts anymore (#11053). Is this still...
I wonder if it is enough to log just the current partition-range and slice. After all, what is interesting to the user is that some partitions have too many tombstones...
(please fix the minor issues I pointed out and I'll merge this PR)