Review Board 1.7.22


SQOOP-1154 Sqoop2: Text partitioner might miss or include edge values

Review Request #13051 - Created July 30, 2013 and updated

Jarek Cecho
SQOOP-1154
Reviewers
Sqoop
sqoop-sqoop2
I've altered the methods to inject the lower bound into first partition and upper bound to the last partition.
Altered one unit test and tested in the new integration test that is part of SQOOP-1114.
Ship it!
Posted (July 30, 2013, 6:12 p.m.)
Good point.   Even though these were in line with what we did before, it is important that we have the bounds to include both.
Posted (July 31, 2013, 4:08 p.m.)
Logical +1.