[PARQUET-1489] Insufficient documentation for UserDefinedPredicate.keep(T)

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.11.0
• Component/s: parquet-mr
• Labels: pull-request-available
• Assignee: Gabor Szadovszky
• Reporter: Gabor Szadovszky
• Created: 09/Jan/19 10:32
• Updated: 09/Jan/19 12:41
• Resolved: 09/Jan/19 12:41

Description

The parquet-mr library invokes UserDefinedPredicate.keep(T) with null values. (This is the only way parquet-mr can check if a null fulfills the UDP.) This fact is not highlighted in the documentation of the method which can easily lead to an unexpected NPE.

Issue Links

• relates to: PARQUET-1488 UserDefinedPredicate throw NPE
• links to: GitHub Pull Request #588