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` .---
