I overwrote the run() method in the mapper with a run() method (below) that
catches the EOFException. The mapper and reducer now complete but the
outputted lzo file from the reducer throws an "Unexpected End of File error"
when decompressing it indicating something did not clean up properly. I
can't think of why this could be happening as the map() method should only
be called on input that was properly decompressed (anything that can't be
decompressed will throw an Exception that is being caught). The reducer
then should not even know that the mapper hit an EOFException in the input
gzip file, and yet the output lzo file still has the unexpected end of file
problem (I'm using the kevinweil lzo libraries). Is there some call that
needs to be made that will close out the mapper and ensure that the lzo
output from the reducer is formatted properly? Thank you!
@Override
public void run(Context context) throw InterruptedException{
try{
setup(context);
while(context.nextKeyValue()){
map(context.getCurrentKey(), context.getCurrentValue(),
context);
}
cleanup(context);
} catch(EOFException){
logError(context, "EOFException: Corrupt gzip file" + mFileName);
}
}
On Thu, Oct 21, 2010 at 1:29 PM, ed <[email protected]> wrote:
> Thanks Tom! Didn't see your post before posting =)
>
>
> On Thu, Oct 21, 2010 at 1:28 PM, ed <[email protected]> wrote:
>
>> Sorry to keep spamming this thread. It looks like the correct way to
>> implement MapRunnable using the new mapreduce classes (instead of the
>> deprecated mapred) is to override the run() method of the mapper class.
>> This is actually nice and convenient since everyone should already be using
>> Mapper class (org.apache.hadoop.mapreduce.Maper<KEYIN, VALUEIN, KEYOUT,
>> VALUEOUT> for their mappers.
>>
>> ~Ed
>>
>>
>> On Thu, Oct 21, 2010 at 12:14 PM, ed <[email protected]> wrote:
>>
>>> Just checked the Hadoop 0.21.0 API docs (I was looking in the wrong docs
>>> before) and it doesn't look like MapRunner is deprecated so I'll try
>>> catching the error there and will report back if it's a good solution.
>>> Thanks!
>>>
>>> ~Ed
>>>
>>>
>>> On Thu, Oct 21, 2010 at 11:23 AM, ed <[email protected]> wrote:
>>>
>>>> Hello,
>>>>
>>>> The MapRunner classes looks promising. I noticed it is in the
>>>> deprecated mapred package but I didn't see an equivalent class in the
>>>> mapreduce package. Is this going to ported to mapreduce or is it no longer
>>>> being supported? Thanks!
>>>>
>>>> ~Ed
>>>>
>>>>
>>>> On Thu, Oct 21, 2010 at 6:36 AM, Harsh J <[email protected]>wrote:
>>>>
>>>>> If it occurs eventually as your record reader reads it, then you may
>>>>> use a MapRunner class instead of a Mapper IFace/Subclass. This way,
>>>>> you may try/catch over the record reader itself, and call your map
>>>>> function only on valid next()s. I think this ought to work.
>>>>>
>>>>> You can set it via JobConf.setMapRunnerClass(...).
>>>>>
>>>>> Ref: MapRunner API @
>>>>>
>>>>> http://hadoop.apache.org/common/docs/r0.20.2/api/org/apache/hadoop/mapred/MapRunner.html
>>>>>
>>>>> On Wed, Oct 20, 2010 at 4:14 AM, ed <[email protected]> wrote:
>>>>> > Hello,
>>>>> >
>>>>> > I have a simple map-reduce job that reads in zipped files and
>>>>> converts them
>>>>> > to lzo compression. Some of the files are not properly zipped which
>>>>> results
>>>>> > in Hadoop throwing an "java.io.EOFException: Unexpected end of input
>>>>> stream
>>>>> > error" and causes the job to fail. Is there a way to catch this
>>>>> exception
>>>>> > and tell hadoop to just ignore the file and move on? I think the
>>>>> exception
>>>>> > is being thrown by the class reading in the Gzip file and not my
>>>>> mapper
>>>>> > class. Is this correct? Is there a way to handle this type of error
>>>>> > gracefully?
>>>>> >
>>>>> > Thank you!
>>>>> >
>>>>> > ~Ed
>>>>> >
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Harsh J
>>>>> www.harshj.com
>>>>>
>>>>
>>>>
>>>
>>
>
