address feedback

Author: gh-yzou

LogicalTypes.md

@@ -521,11 +521,11 @@ as shown below.
     </tr>
 </table>
 
-### INTERVAL types
+### Interval types
 
 #### INTERVAL
 `INTERVAL` is *deprecated*. Please use `INTERVAL_YEAR_MONTH` and `INTERVAL_DAY_TIME`
-as a more precise representation per [ANSI SQL Standard](https://www.ibm.com/docs/en/informix-servers/14.10.0?topic=types-ansi-sql-standards-datetime-interval-values).
+as a more precise representation per [ANSI SQL Standard](https://docs.oracle.com/en/database/oracle/oracle-database/19/sqlrf/Data-Types.html#GUID-7690645A-0EE3-46CA-90DE-C96DF5A01F8F).
 
 `INTERVAL` is used for an interval of time. It must annotate a
 `fixed_len_byte_array` of length 12. This array stores three little-endian
@@ -544,19 +544,21 @@ statistics should be saved for this type and if such non-compliant statistics
 are found during reading, they must be ignored.
 
 #### INTERVAL_YEAR_MONTH
+
 `INTERVAL_YEAR_MONTH` is used to represent a year-month time interval, such as
 `4 years and 6 months`. It must annotate an `int32` that stores the total number
 of months as a signed integer, which represents the interval and can be negative.
 The time duration is independent of any timezone.
 
 #### INTERVAL_DAY_TIME
+
 `INTERVAL_DAY_TIME` is used to represent a day-time time interval, such as
 `5 days, 10 hours and 30 minutes`. It must annotate a 16-byte `FIXED_LEN_BYTE_ARRAY`
 that stores the total number of nanoseconds representing the interval. The value is
 a signed integer and can be negative to indicate backward duration.
 The time interval is independent of any timezone.
 
-Based on the [ANSI SQL standard](https://web.cecs.pdx.edu/~len/sql1999.pdf) definition of the INTERVAL data type and fields values, 
+Based on the [ANSI SQL standard](https://docs.oracle.com/en/database/oracle/oracle-database/19/sqlrf/Data-Types.html#GUID-7690645A-0EE3-46CA-90DE-C96DF5A01F8F) definition of the INTERVAL data type and fields values, 
 an interval of 1 day is equivalent to 24 hours, regardless of the specific number of 
 seconds in a day. This means that when you define an interval of `1 day`, 
 it represents exactly 24 hours, or 86,400,000,000 nanoseconds.

src/main/thrift/parquet.thrift

@@ -1098,6 +1098,8 @@ union ColumnOrder {
    *   VARIANT - undefined
    *   GEOMETRY - undefined
    *   GEOGRAPHY - undefined
+   *   INTERVAL_YEAR_MONTH - signed comparison of the represented value
+   *   INTERVAL_DAY_TIME - signed comparison of the represented value
    *
    * In the absence of logical types, the sort order is determined by the physical type:
    *   BOOLEAN - false, true