[
https://issues.apache.org/jira/browse/DRILL-5936?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16247941#comment-16247941
]
ASF GitHub Bot commented on DRILL-5936:
---------------------------------------
Github user amansinha100 commented on the issue:
https://github.com/apache/drill/pull/1025
+1 with a minor comment. In the commit message and JIRA it would be
better to say 'code inspection' instead of code review which may be
interpreted to mean the normal code review process.
> Refactor MergingRecordBatch based on code review
> ------------------------------------------------
>
> Key: DRILL-5936
> URL: https://issues.apache.org/jira/browse/DRILL-5936
> Project: Apache Drill
> Issue Type: Improvement
> Components: Tools, Build & Test
> Reporter: Vlad Rozov
> Assignee: Vlad Rozov
> Priority: Minor
>
> * Reorganize code to remove unnecessary {{pqueue.peek()}}
> * Reuse Node
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
