[GitHub] drill issue #860: DRILL-5601: Rollup of external sort fixes and improvements

2017-08-14 Thread jinfengni
Github user jinfengni commented on the issue: https://github.com/apache/drill/pull/860 +1 LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

Re: direct.used on the Metrics page exceeded planner.memory.max_query_memory_per_node while running a single query

2017-08-14 Thread Boaz Ben-Zvi
There is this page: https://drill.apache.org/docs/sort-based-and-hash-based-memory-constrained-operators/ But it seems out of date (correct for 1.10). It does not explain about the hash operators, except that they run till they can not allocate any more memory. This will happen (undocumented) at

Query Hanging in Running State

2017-08-14 Thread Sorabh Hamirwasia
Hi All, Recently I found an issue (Thanks to knguyen to help with test setup) related to Fragment Status reporting and would like some feedback on it. When a client submits a query to Foreman, then it is planned by Foreman and later fragments are scheduled to root and non-root nodes. Foreman c

[jira] [Created] (DRILL-5722) Query with root and non-root fragments can potentially hang if Control Connection fails due to network issue.

2017-08-14 Thread Sorabh Hamirwasia (JIRA)
Sorabh Hamirwasia created DRILL-5722: Summary: Query with root and non-root fragments can potentially hang if Control Connection fails due to network issue. Key: DRILL-5722 URL: https://issues.apache.org/jira/

[jira] [Created] (DRILL-5721) Query with only root fragment and no non-root fragment hangs when Drillbit to Drillbit Control Connection has network issues

2017-08-14 Thread Sorabh Hamirwasia (JIRA)
Sorabh Hamirwasia created DRILL-5721: Summary: Query with only root fragment and no non-root fragment hangs when Drillbit to Drillbit Control Connection has network issues Key: DRILL-5721 URL: https://issues.a

Re: direct.used on the Metrics page exceeded planner.memory.max_query_memory_per_node while running a single query

2017-08-14 Thread Muhammad Gelbana
I'm not sure which version I was using when that happened. But that's some precise details you've mentioned! Is this mentioned somewhere in the docs ? Thanks a lot. On Aug 14, 2017 9:00 PM, "Boaz Ben-Zvi" wrote: > Did your query include a hash join ? > > As of 1.11, only the External Sort and H

Re: direct.used on the Metrics page exceeded planner.memory.max_query_memory_per_node while running a single query

2017-08-14 Thread Boaz Ben-Zvi
Did your query include a hash join ? As of 1.11, only the External Sort and Hash Aggregate operators obey the memory limit (that is, the “max query memory per node” figure is divided among all the instances of these operators). The Hash Join (as was before 1.11) still does not take part in this

[GitHub] drill issue #858: DRILL-3640: Support JDBC Statement.setQueryTimeout(int)

2017-08-14 Thread laurentgo
Github user laurentgo commented on the issue: https://github.com/apache/drill/pull/858 I think you can actually do 2 and 3 with the same approach, depending on how you compute the remaining time. --- If your project is set up for it, you can reply to this email and have your reply ap

Re: [GitHub] drill pull request #904: DRILL-5717: let some date time test cases be Local ...

2017-08-14 Thread weijie tong
Thanks for the advice ,will consider that. On Mon, 14 Aug 2017 at 7:56 PM vvysotskyi wrote: > Github user vvysotskyi commented on a diff in the pull request: > > https://github.com/apache/drill/pull/904#discussion_r132932858 > > --- Diff: > exec/java-exec/src/main/codegen/templates/DateI

[GitHub] drill pull request #904: DRILL-5717: let some date time test cases be Local ...

2017-08-14 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/904#discussion_r132932858 --- Diff: exec/java-exec/src/main/codegen/templates/DateIntervalFunctionTemplates/DateToCharFunctions.java --- @@ -65,7 +65,7 @@ public void setup() {

[GitHub] drill pull request #904: DRILL-5717: let some date time test cases be Local ...

2017-08-14 Thread vvysotskyi
Github user vvysotskyi commented on a diff in the pull request: https://github.com/apache/drill/pull/904#discussion_r132930206 --- Diff: contrib/format-maprdb/pom.xml --- @@ -75,6 +75,12 @@ com.mapr.fs mapr-hbase + + +