Opened 9 years ago

Closed 8 years ago

Last modified 6 years ago

#25809 closed New feature (fixed)

PostgreSQL 9.5 BRIN Index support in contrib.postgres

Reported by: Asif Saifuddin Auvi Owned by: Mads Jensen
Component: contrib.postgres Version: dev
Severity: Normal Keywords: postgres, index, BRIN, db-indexes
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

pgsql 9.5 introduces BRIN index which will be nice to have in django postgres.

Change History (18)

comment:1 by Tim Graham, 9 years ago

Do you have a proposed API?

in reply to:  1 comment:2 by Asif Saifuddin Auvi, 9 years ago

Replying to timgraham:

Do you have a proposed API?

not till now, but should be based on Class based Index Dep?

comment:3 by Tim Graham, 9 years ago

Triage Stage: Unreviewed → Someday/Maybe

I'd guess so.

comment:4 by Tim Graham, 8 years ago

Triage Stage: Someday/Maybe → Accepted

This can likely move forward now that the framework for class-based indexes is in master.

in reply to:  4 comment:5 by Asif Saifuddin Auvi, 8 years ago

Replying to timgraham:

This can likely move forward now that the framework for class-based indexes is in master.

exactly. will this be part of current gsoc?

Last edited 8 years ago by Asif Saifuddin Auvi (previous) (diff)

comment:6 by Asif Saifuddin Auvi, 8 years ago

Keywords: postgres index BRIN added

comment:7 by Mads Jensen, 8 years ago

Has patch: set
Patch needs improvement: set

​PR

Last edited 8 years ago by Tim Graham (previous) (diff)

comment:8 by Mads Jensen, 8 years ago

Owner: set to Mads Jensen
Status: new → assigned

comment:9 by Mads Jensen, 8 years ago

Patch needs improvement: unset

Updated according to review by Tim Graham. It's tricky finding a good sql_create_index that suits all backends, as the current design requires. My solution just introduces a new one, and uses this in the as_sql-method.

Last edited 8 years ago by Mads Jensen (previous) (diff)

comment:10 by Markus Holtermann, 8 years ago

Triage Stage: Accepted → Ready for checkin

comment:11 by Mads Jensen <mje@…>, 8 years ago

Resolution: → fixed
Status: assigned → closed

In e585c43b:

Fixed #25809 -- Added BrinIndex support in django.contrib.postgres.

Thanks Tim Graham and Markus Holtermann for review.

comment:12 by Akshesh Doshi, 8 years ago

Keywords: db-indexes added

comment:13 by Markus Holtermann <info@…>, 8 years ago

In 7e299c0:

Refs #25809 -- Prefered imports from django.db.models.

comment:14 by Tim Graham <timograham@…>, 8 years ago

In 65e321b7:

Refs #25809 -- Made a few late review comments for BrinIndex.

comment:15 by GitHub <noreply@…>, 7 years ago

In d1d7348:

Refs #25809 -- Removed BrinIndex.repr().

In retrospect, implementing a repr() for index subclasses doesn't
seem worthwhile.

comment:16 by Tim Graham <timograham@…>, 7 years ago

In fb42d024:

Refs #25809 -- Omitted pages_per_range from BrinIndex.deconstruct() if it's None.

comment:17 by Tim Graham <timograham@…>, 7 years ago

In c685b8f8:

[1.11.x] Refs #25809 -- Omitted pages_per_range from BrinIndex.deconstruct() if it's None.

Backport of fb42d0247136249ea81962474e9a6a2faf1755f1 from master

comment:18 by Tim Graham <timograham@…>, 6 years ago

In ff9543b:

Refs #25809, #28990 -- Added PostgreSQL version check for BrinIndex support.

Note: See TracTickets for help on using tickets.
Back to Top