[PIG-2132] Piggybank: MIN and MAX functions should ignore nulls

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.10.1
• Fix Version/s: 0.13.0
• Component/s: tools
• Labels: newbie piggybank

Release Note

All I did was go through all of the Max and Min functions in Piggybank and add
if (first==null)
    return second;
if (second==null)
    return first;

Following conversation on the message board, the desired outputs are:
null,null -> null
1,null -> 1
null,1 -> 1
1,1 -> 1

This change gives that.

• Tags: piggybank
• Assignee: Rekha Joshi
• Reporter: Jonathan Coveney
• Created: 17/Jun/11 17:07
• Updated: 07/Jul/14 18:08
• Resolved: 27/Nov/13 21:21

Description

org.apache.pig.piggybank.evaluation.math.MAX and org.apache.pig.piggybank.evaluation.math.MIN throws a NullPointerException on a null input, when it should just ignore it.