Skip to content

Bug fix for multi-column indexes#1347

Merged
zachmu merged 15 commits intomainfrom
zachmu/django
Apr 2, 2025
Merged

Bug fix for multi-column indexes#1347
zachmu merged 15 commits intomainfrom
zachmu/django

Conversation

@zachmu
Copy link
Copy Markdown
Member

@zachmu zachmu commented Mar 31, 2025

Fixes #1331

Also fixes an encoding bug: limited length varchar columns were always being encoded out of band.

Tests for the changes in dolthub/dolt#9061

Also fixes a bug in parsing --prof args

@zachmu zachmu requested a review from Hydrocharged March 31, 2025 23:25
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2025

Main PR
covering_index_scan_postgres 352.43/s 344.57/s -2.3%
index_join_postgres 153.35/s 153.67/s +0.2%
index_join_scan_postgres 182.43/s 184.86/s +1.3%
index_scan_postgres 12.57/s 12.97/s +3.1%
oltp_point_select 2501.14/s 2613.15/s +4.4%
oltp_read_only 1731.23/s ${\color{red}726.69/s}$ ${\color{red}-58.1\%}$
select_random_points 107.72/s 117.85/s +9.4%
select_random_ranges 130.85/s ${\color{red}8.72/s}$ ${\color{red}-93.4\%}$
table_scan_postgres 10.55/s ${\color{lightgreen}12.08/s}$ ${\color{lightgreen}+14.5\%}$
types_table_scan_postgres 5.25/s 5.57/s +6.0%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2025

Main PR
Total 42090 42090
Successful 15573 15582
Failures 26517 26508
Partial Successes1 5208 5208
Main PR
Successful 36.9993% 37.0207%
Failures 63.0007% 62.9793%

${\color{lightgreen}Progressions (9)}$

gin

QUERY: analyze t_gin_test_tbl;

join

QUERY: analyze tt3;
QUERY: analyze tt4;

partition_join

QUERY: analyze prtx1;
QUERY: analyze prtx2;

stats_ext

QUERY: ANALYZE tststats.priv_test_tbl;

subselect

QUERY: analyze exists_tbl;

vacuum

QUERY: ANALYZE pg_catalog.pg_class;
QUERY: ANALYZE pg_catalog.pg_authid;

Footnotes

  1. These are tests that we're marking as Successful, however they do not match the expected output in some way. This is due to small differences, such as different wording on the error messages, or the column names being incorrect while the data itself is correct.

@zachmu zachmu enabled auto-merge April 2, 2025 16:49
@zachmu zachmu merged commit c59f4d5 into main Apr 2, 2025
14 checks passed
@zachmu zachmu deleted the zachmu/django branch April 2, 2025 17:41
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.

Django requires CREATE INDEX with what looks like a regex pattern

2 participants