emkornfield commented on a change in pull request #11310:
URL: https://github.com/apache/arrow/pull/11310#discussion_r724668988
##########
File path: go/arrow/datatype.go
##########
@@ -95,21 +95,30 @@ const (
// nanoseconds since midnight
TIME64
- // INTERVAL is YEAR_MONTH or DAY_TIME interval in SQL style
- INTERVAL
+ // INTERVAL_MONTHS is YEAR_MONTH in SQL style
+ INTERVAL_MONTHS
- // DECIMAL is a precision- and scale-based decimal type. Storage type
depends on the
+ // INTERVAL_DAY_TIME is DAY_TIME in SQL style
+ INTERVAL_DAY_TIME
+
+ // DECIMAL128 is a precision- and scale-based decimal type. Storage
type depends on the
// parameters.
- DECIMAL
Review comment:
can DECIMAL be made an alias for DECIMAL128 to avoid breaking any
consumers?
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
