[GitHub] drill pull request #1045: DRILL-5730 Test Mocking Improvements

[paul-rogers]

Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1045#discussion_r162229362
  
    --- Diff: 
contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/HiveUtilities.java
 ---
    @@ -288,7 +288,7 @@ public static void populateVector(final ValueVector 
vector, final DrillBuf manag
         }
       }
     
    -  public static MajorType getMajorTypeFromHiveTypeInfo(final TypeInfo 
typeInfo, final OptionManager options) {
    +  public static MajorType getMajorTypeFromHiveTypeInfo(final TypeInfo 
typeInfo, final OptionSet options) {
    --- End diff --
    
    Note that DRILL-6049 removes OptionSet in favor of OptionManager. (Thanks 
to the work you and Jyothsna did, we can now use that class in tests.)


---