Re: Changing behavior on trunk

2017-08-02 Thread Anna Szonyi
Hi All, As no one voiced any concerns, I will commit the change tomorrow morning. Thanks, Anna On Wed, Jul 26, 2017 at 2:36 PM, Boglarka Egyed wrote: > Hi All, > > +1 to Szabi's comment. As it is a bugfix I wouldn't think that a new option > should be introduced to switch

Re: Changing behavior on trunk

2017-07-26 Thread Boglarka Egyed
Hi All, +1 to Szabi's comment. As it is a bugfix I wouldn't think that a new option should be introduced to switch it on and off. However, it should be emphasized in the release notes (in whichever release it will be included) that this is a behavior changing fix. Thanks for bringing it up Anna!

Re: Changing behavior on trunk

2017-07-21 Thread Szabolcs Vasas
Hi guys, I think that this is basically a bug fix and because of that I would not add a new flag to switch it on/off. It is true that it changes behavior but many other bug fixes do the same so if we add a switch now we might want to do the same thing later and that could lead to a large number

Changing behavior on trunk

2017-07-20 Thread Anna Szonyi
Hi @dev list, Jilani has created a great patch for SQOOP-3149 : Sqoop incremental import - NULL column updates are not pulled into HBase table. This is a patch that (in my opinion) corrects previously incorrect behavior in sqoop, however it can