Change History (12)

comment:1 Changed 19 months ago by Carlton Gibson

This is how Postres ranges are specified. See https://www.postgresql.org/docs/9.3/rangetypes.html#RANGETYPES-IO

Wonder if we can explain that better.

comment:2 Changed 19 months ago by Mariusz Felisiak

Yes, see https://www.postgresql.org/docs/current/rangetypes.html#RANGETYPES-INCLUSIVITY.

We can only add ref to the PostgreSQL documentation like in RangeBoundary.

comment:3 Changed 19 months ago by Carlton Gibson

Resolution: wontfix
Status: newclosed

Hmmm. We already link to the psycopg docs here: https://www.psycopg.org/docs/extras.html#adapt-range.

I'm going to say wontfix, as the [) is already marked out as a code literal. But if anyone wants to suggest a concrete clarification I'm happy to review.

comment:4 in reply to:  3 Changed 19 months ago by Joon Hwan 김준환

I had difficulty reading the document because I didn't have any background knowledge of Range Boundary expression.
I carefully suggests putting the next link nearby Range Fields.
https://docs.djangoproject.com/en/3.0/ref/contrib/postgres/fields/#rangeboundary-expressions
Or It would be nice if there was an explanation for whether [) is a Range Boundary expression.

If you don't mind, I will request PR.

Last edited 19 months ago by Joon Hwan 김준환 (previous) (diff)

comment:5 Changed 19 months ago by Carlton Gibson

A PR clarifying would be very welcome. Please assign yourself and reopen the ticket if you will provide one. Thanks!

comment:6 Changed 19 months ago by Joon Hwan 김준환

Resolution: wontfix
Status: closednew

Let me provide PR as soon as possible.

comment:7 Changed 19 months ago by Joon Hwan 김준환

Owner: changed from nobody to Joon Hwan 김준환
Status: newassigned

comment:8 Changed 19 months ago by Carlton Gibson

Triage Stage: UnreviewedAccepted

OK, super. I shall Accept on that basis.

Thanks for the input! 🎉

comment:9 Changed 19 months ago by Joon Hwan 김준환

Has patch: set

comment:10 Changed 19 months ago by Mariusz Felisiak

Triage Stage: AcceptedReady for checkin

comment:11 Changed 19 months ago by Mariusz Felisiak <felisiak.mariusz@…>

Resolution: fixed
Status: assignedclosed

In 4029bcd6:

Fixed #31577 -- Clarified docs about bounds of RangeFields.

comment:12 Changed 19 months ago by Mariusz Felisiak <felisiak.mariusz@…>

In 03bfcba:

[3.1.x] Fixed #31577 -- Clarified docs about bounds of RangeFields.

Backport of 4029bcd6b20f75a78f9a5829d7826c79aeb20732 from master

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