Re: Review Request: Cleaned up error codes in MapreduceExecutionError

2013-03-28 Thread Kathleen Ting
On Feb. 18, 2013, 8:31 p.m., Linden Hillenbrand wrote: > > Jarcec > > Linden Hillenbrand wrote: > Hey Jarcec, > > I have a quick question (just want to make sure I go about the > renumeration correctly), therefore when I renumerate I need to change the > error code references in t

Re: Review Request: Cleaned up error codes in MapreduceExecutionError

2013-03-02 Thread Linden Hillenbrand
On Feb. 18, 2013, 8:31 p.m., Linden Hillenbrand wrote: > > Jarcec Hey Jarcec, I have a quick question (just want to make sure I go about the renumeration correctly), therefore when I renumerate I need to change the error code references in the codebase so they call the proper exceptions. When

Re: Review Request: Cleaned up error codes in MapreduceExecutionError

2013-03-02 Thread Linden Hillenbrand
> On Feb. 18, 2013, 8:31 p.m., Jarek Cecho wrote: > > execution/mapreduce/src/main/java/org/apache/sqoop/job/MapreduceExecutionError.java, > > line 27 > > > > > > Would you mind keeping this around? I know that it's unu

Re: Review Request: Cleaned up error codes in MapreduceExecutionError

2013-02-18 Thread Jarek Cecho
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9495/#review16714 --- Thank you Linden for taking a look into this! I appreciate your time.