Github user fszabo2 commented on a diff in the pull request:

    https://github.com/apache/sqoop/pull/60#discussion_r237881838
  
    --- Diff: 
src/test/org/apache/sqoop/importjob/configuration/MysqlImportJobTestConfiguration.java
 ---
    @@ -65,4 +66,21 @@
       public String toString() {
         return getClass().getSimpleName();
       }
    +
    +  @Override
    +  public Object[] getExpectedResultsForHive() {
    --- End diff --
    
    Hi Attila, 
    
    I'd like to leave these as is.
    
    I'd argue that these don't actually belong together, so it's not really 
code duplication but rather an accidental match.
    
    I added a new column to the sample data of the mysql configuration to cover 
an edge case. The expected result in Hive changed for this configuration, and, 
on the other hand this edge case does not make sense to be added to the SQL 
server tests.
    



---

Reply via email to