[GitHub] drill pull request #1190: [DRILL-5937] ExecConstants: changed comment, timeo...

2018-03-26 Thread pushpendra-jaiswal-90
GitHub user pushpendra-jaiswal-90 opened a pull request: https://github.com/apache/drill/pull/1190 [DRILL-5937] ExecConstants: changed comment, timeout default is 30 sec. **Changed comment** to reflect default prepared statement timeout as 30 sec. (same as hive) . 10 seconds

[GitHub] drill issue #1190: [DRILL-5937] ExecConstants: changed comment, timeout defa...

2018-03-26 Thread pushpendra-jaiswal-90
Github user pushpendra-jaiswal-90 commented on the issue: https://github.com/apache/drill/pull/1190 @vrozov @vdiravka Could you please review this? ---

[GitHub] drill issue #1190: DRILL-5937: ExecConstants: changed comment, timeout defau...

2018-03-26 Thread pushpendra-jaiswal-90
Github user pushpendra-jaiswal-90 commented on the issue: https://github.com/apache/drill/pull/1190 @arina-ielchiieva I've done the required changes. ---

[GitHub] drill issue #1190: DRILL-5937: ExecConstants: changed comment, timeout defau...

2018-03-26 Thread pushpendra-jaiswal-90
Github user pushpendra-jaiswal-90 commented on the issue: https://github.com/apache/drill/pull/1190 @arina-ielchiieva Can you please tell me which drill-module.conf has it ? ---

[GitHub] drill issue #1190: DRILL-5937: ExecConstants: changed comment, timeout defau...

2018-03-27 Thread pushpendra-jaiswal-90
Github user pushpendra-jaiswal-90 commented on the issue: https://github.com/apache/drill/pull/1190 @arina-ielchiieva I have done the changes. Could you please review? ---

[GitHub] drill issue #1190: DRILL-5937: ExecConstants: changed comment, timeout defau...

2018-03-27 Thread pushpendra-jaiswal-90
Github user pushpendra-jaiswal-90 commented on the issue: https://github.com/apache/drill/pull/1190 @arina-ielchiieva I have removed comment. ---

[GitHub] drill issue #1190: DRILL-5937: ExecConstants: changed comment, timeout defau...

2018-03-27 Thread pushpendra-jaiswal-90
Github user pushpendra-jaiswal-90 commented on the issue: https://github.com/apache/drill/pull/1190 @vrozov @vdiravka Could you please review this? ---