[HIVE-9322] Make null-checks consistent for MapObjectInspector subclasses.

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.14.0
• Fix Version/s: 1.2.0
• Component/s: Serializers/Deserializers
• Assignee: Mithun Radhakrishnan
• Reporter: Mithun Radhakrishnan
• Created: 09/Jan/15 01:10
• Updated: 18/May/15 19:56
• Resolved: 20/Feb/15 23:12

Description

LazyBinaryMapObjectInspector, DeepParquetHiveMapInspector, etc. check both the map-column value and the map-key for null, before dereferencing them. OrcMapObjectInspector and LazyMapObjectInspector do not.

This patch brings them all in sync. Might not be a real problem, unless (for example) the lookup key is itself a (possibly null) value from another column.