[ https://issues.apache.org/jira/browse/PIG-2132?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13051348#comment-13051348 ]
Jonathan Coveney commented on PIG-2132: --------------------------------------- Dmitriy, I will definitely do that. A question on style: is it better to contain all tests of related functions to one function (in this case, textMAX() and testMIN()), or should I break it out? > Piggybank: MIN and MAX functions should ignore nulls > ---------------------------------------------------- > > Key: PIG-2132 > URL: https://issues.apache.org/jira/browse/PIG-2132 > Project: Pig > Issue Type: Bug > Components: tools > Reporter: Jonathan Coveney > Priority: Minor > Attachments: JcoMaxMinPatch.patch > > Original Estimate: 4h > Remaining Estimate: 4h > > 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. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira