[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r341198541 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r341087581 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r341086620 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r341084431 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r341084431 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r341083184 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339670241 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339669590 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339666167 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339665919 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339439943 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339438220 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339436925 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339436343 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339435212 ## File path: sql/catalyst/src/main/java/org/apache/spark/sql/cataly

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339434784 ## File path: sql/catalyst/src/main/java/org/apache/spark/sql/cataly

[GitHub] [spark] cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds

cloud-fan commented on a change in pull request #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#discussion_r339433234 ## File path: common/unsafe/src/main/java/org/apache/spark/unsafe/ty