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 2018/01/17 06:54:20 UTC

[GitHub] drill pull request #1093: DRILL-6093 : Account for simple columns in project...

GitHub user gparai opened a pull request:

    https://github.com/apache/drill/pull/1093

    DRILL-6093 : Account for simple columns in project cpu costing

    @amansinha100 can you please review this PR? Thanks!

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/gparai/drill DRILL-6093-master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/drill/pull/1093.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 #1093
    
----
commit 30f1934e6b57aeaf33f72ac701bd431f2c11e403
Author: Gautam Parai <gp...@...>
Date:   2018-01-16T23:16:16Z

    DRILL-6093 : Account for simple columns in project cpu costing

----


---

[GitHub] drill pull request #1093: DRILL-6093 : Account for simple columns in project...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/drill/pull/1093


---

[GitHub] drill issue #1093: DRILL-6093 : Account for simple columns in project cpu co...

Posted by amansinha100 <gi...@git.apache.org>.

Github user amansinha100 commented on the issue:

    https://github.com/apache/drill/pull/1093
  
    +1 .  It would be good to add a unit test using the example in the JIRA. 


---

[GitHub] drill issue #1093: DRILL-6093 : Account for simple columns in project cpu co...

Posted by gparai <gi...@git.apache.org>.

Github user gparai commented on the issue:

    https://github.com/apache/drill/pull/1093
  
    @amansinha100 I have addressed your comments.


---