Copilot commented on code in PR #53239:
URL: https://github.com/apache/arrow/pull/53239#discussion_r4242630329
##########
docs/source/format/CanonicalExtensions.rst:
##########
@@ -617,6 +617,217 @@ This extension type is intended to be compatible with
ANSI SQL's ``TIMESTAMP WIT
It is also *permissible* for the ``offset_minutes`` field to be
dictionary-encoded or run-end-encoded.
+.. _fixed_closedness_range_extension:
+
+Fixed closedness range
+======================
+
+Fixed closedness range represents a bounded set (mathematical interval)
Review Comment:
This describes the type as a “bounded set,” but the storage contract below
explicitly permits one or both bounds to be null and defines `(-inf, +inf)`.
Those are unbounded ranges, so the introduction misstates the type's domain.
Please describe it as a range with optional lower and upper bounds and update
the repeated “bounded set” wording in both sections.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
