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

    https://github.com/apache/incubator-hivemall/pull/124#discussion_r148199276
  
    --- Diff: core/src/main/java/hivemall/tools/list/UDAFToOrderedList.java ---
    @@ -404,11 +407,9 @@ void merge(@Nonnull List<Object> o_keyList, @Nonnull 
List<Object> o_valueList) {
                     }
                 }
     
    -            @Nonnull
                 Pair<List<Object>, List<Object>> drainQueue() {
    --- End diff --
    
    Add `@Nullable` for the return of `drainQueue` .


---

Reply via email to