You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by gparai <gi...@git.apache.org> on 2017/10/07 02:02:45 UTC
[GitHub] drill pull request #979: DRILL-5853 : Update Calcite to get NULL direction f...
GitHub user gparai opened a pull request:
https://github.com/apache/drill/pull/979
DRILL-5853 : Update Calcite to get NULL direction for sort removal
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gparai/drill Drill-5853-ACM
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/drill/pull/979.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #979
----
commit 89b01d45c5d26d4cc1f20d75fd26d25fc5765bc5
Author: Gautam Parai <gp...@maprtech.com>
Date: 2017-10-07T01:48:42Z
DRILL-5853 : Update Calcite to get NULL direction for sort removal
----
---
[GitHub] drill issue #979: DRILL-5853 : Update Calcite to get NULL direction for sort...
Posted by paul-rogers <gi...@git.apache.org>.
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/979
@amansinha100, this is a one-liner; do you have time for a quick review? Thanks.
---
[GitHub] drill pull request #979: DRILL-5853 : Update Calcite to get NULL direction f...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/drill/pull/979
---
[GitHub] drill issue #979: DRILL-5853 : Update Calcite to get NULL direction for sort...
Posted by amansinha100 <gi...@git.apache.org>.
Github user amansinha100 commented on the issue:
https://github.com/apache/drill/pull/979
+1
---
[GitHub] drill issue #979: DRILL-5853 : Update Calcite to get NULL direction for sort...
Posted by gparai <gi...@git.apache.org>.
Github user gparai commented on the issue:
https://github.com/apache/drill/pull/979
@amansinha100 Can you please review the PR? Thanks!
---