[jira] Updated: (PIG-1088) change merge join and merge join indexer to work with new LoadFunc interface
[ https://issues.apache.org/jira/browse/PIG-1088?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alan Gates updated PIG-1088: Fix Version/s: 0.7.0 > change merge join and merge join indexer to work with new LoadFunc interface > > > Key: PIG-1088 > URL: https://issues.apache.org/jira/browse/PIG-1088 > Project: Pig > Issue Type: Sub-task >Reporter: Thejas M Nair >Assignee: Thejas M Nair > Fix For: 0.7.0 > > Attachments: PIG-1088.1.patch, PIG-1088.patch > > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-1088) change merge join and merge join indexer to work with new LoadFunc interface
[ https://issues.apache.org/jira/browse/PIG-1088?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Pradeep Kamath updated PIG-1088: Resolution: Fixed Hadoop Flags: [Incompatible change, Reviewed] Status: Resolved (was: Patch Available) +1, Patch committed to load-store-redesign with the minor change made in consultation with Thejas: DataType.isAtomic returns true for GENERIC_WRITABLECOMPARABLE and DataType.isComplex returns false for it. > change merge join and merge join indexer to work with new LoadFunc interface > > > Key: PIG-1088 > URL: https://issues.apache.org/jira/browse/PIG-1088 > Project: Pig > Issue Type: Sub-task >Reporter: Thejas M Nair >Assignee: Thejas M Nair > Attachments: PIG-1088.1.patch, PIG-1088.patch > > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-1088) change merge join and merge join indexer to work with new LoadFunc interface
[ https://issues.apache.org/jira/browse/PIG-1088?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thejas M Nair updated PIG-1088: --- Attachment: PIG-1088.1.patch Changes address Pradeep's comments. All mergejoin test cases pass. Also ran test-commit test cases and ensured that they match results seen in PIG-1094 . test-patch results - [exec] -1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] -1 tests included. The patch doesn't appear to include any new or modified tests. [exec] Please justify why no tests are needed for this patch. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. > change merge join and merge join indexer to work with new LoadFunc interface > > > Key: PIG-1088 > URL: https://issues.apache.org/jira/browse/PIG-1088 > Project: Pig > Issue Type: Sub-task >Reporter: Thejas M Nair >Assignee: Thejas M Nair > Attachments: PIG-1088.1.patch, PIG-1088.patch > > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-1088) change merge join and merge join indexer to work with new LoadFunc interface
[ https://issues.apache.org/jira/browse/PIG-1088?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thejas M Nair updated PIG-1088: --- Status: Patch Available (was: Open) Patch is for load-store-redesign branch, will not work on trunk. > change merge join and merge join indexer to work with new LoadFunc interface > > > Key: PIG-1088 > URL: https://issues.apache.org/jira/browse/PIG-1088 > Project: Pig > Issue Type: Sub-task >Reporter: Thejas M Nair >Assignee: Thejas M Nair > Attachments: PIG-1088.patch > > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-1088) change merge join and merge join indexer to work with new LoadFunc interface
[ https://issues.apache.org/jira/browse/PIG-1088?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thejas M Nair updated PIG-1088: --- Attachment: PIG-1088.patch * LoadOrderedInput (proposed) is called OrderedLoadFunc . It is an abstract class because LoadFunc is an abstract class. * TextInputOrder(proposed) is called FileInputLoadFunc. * New internal type called GENERIC_WRITABLECOMPARABLE has been added, to be used for WritableComparable classes. Tuples can read/write this type. * ReadToEndLoader takes a list of input splits to be read All TestMergeJoin test cases are passing. testpatch results - [exec] [exec] -1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] -1 tests included. The patch doesn't appear to include any new or modified tests. [exec] Please justify why no tests are needed for this patch. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. > change merge join and merge join indexer to work with new LoadFunc interface > > > Key: PIG-1088 > URL: https://issues.apache.org/jira/browse/PIG-1088 > Project: Pig > Issue Type: Sub-task >Reporter: Thejas M Nair >Assignee: Thejas M Nair > Attachments: PIG-1088.patch > > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.