[ORC-790] TIMESTAMP_INSTANT should be primitive

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.7.0, 1.6.7
• Fix Version/s: 1.7.0, 1.6.8
• Assignee: Zoltán Borók-Nagy
• Reporter: Zoltán Borók-Nagy
• Created: 26/Apr/21 14:41
• Updated: 21/Sep/21 20:45
• Resolved: 28/Apr/21 17:49

Description

Currently TIMESTAMP_INSTANT is defined as a non-primitive type. While TIMESTAMP is a primitive type.

https://github.com/apache/orc/blob/cfa749854fd4fd78c763762f3515f33b9a206072/java/core/src/java/org/apache/orc/TypeDescription.java#L119

TIMESTAMP_INSTANT("timestamp with local time zone", false);
Category(String name, boolean isPrimitive) {
  this.name = name;
  this.isPrimitive = isPrimitive;
}

This can cause problems in code that branches based on type.getCategory().isPrimitive()

Issue Links

• links to: GitHub Pull Request #691