Skip to content

[statspro] stats ignore non prolly indexes#8969

Merged
max-hoffman merged 1 commit intomainfrom
max/stats-idx-panics
Mar 11, 2025
Merged

[statspro] stats ignore non prolly indexes#8969
max-hoffman merged 1 commit intomainfrom
max/stats-idx-panics

Conversation

@max-hoffman
Copy link
Copy Markdown
Contributor

Stats collection skips non-prolly indexes. A more complete/safe refactoring of index use is in the new stats PR.

fixes: #8964

@coffeegoddd
Copy link
Copy Markdown
Contributor

@max-hoffman DOLT

comparing_percentages
100.000000 to 100.000000
version result total
23386fd ok 5937457
version total_tests
23386fd 5937457
correctness_percentage
100.0

Copy link
Copy Markdown
Contributor

@jycor jycor left a comment

Choose a reason for hiding this comment

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

LGTM

@max-hoffman max-hoffman force-pushed the max/stats-idx-panics branch from 23386fd to 596376e Compare March 11, 2025 18:48
@coffeegoddd
Copy link
Copy Markdown
Contributor

@max-hoffman DOLT

comparing_percentages
100.000000 to 100.000000
version result total
596376e ok 5937457
version total_tests
596376e 5937457
correctness_percentage
100.0

@max-hoffman max-hoffman merged commit 13ee830 into main Mar 11, 2025
20 of 21 checks passed
@max-hoffman max-hoffman deleted the max/stats-idx-panics branch March 11, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Recurrent Panic with querying geometry types after some queries

3 participants