Re: [DISCUSS] Drill 1.12.0 release

2017-10-20 Thread Paul Rogers
Hi Arina, Though I would very much like to get DRILL-5832 and DRILL-5842 into master, I realize everyone is busy and there may not be time to review them. They can wait until after the release if necessary since they are primarily required for a 1.13 project. Thanks, - Paul > On Oct 20,

[GitHub] drill pull request #1005: DRILL-5896: Handle HBase columns vector creation i...

2017-10-20 Thread prasadns14
GitHub user prasadns14 opened a pull request: https://github.com/apache/drill/pull/1005 DRILL-5896: Handle HBase columns vector creation in the HBaseRecordReader Handling vector creation for projected hbase columns, by keeping track of column families and columns separately.

[GitHub] drill issue #1004: DRILL-5876: Use openssl profile to include netty-tcnative...

2017-10-20 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/1004 +1 . Thanks Vlad. Maven awesomeness ! ---

[jira] [Created] (DRILL-5896) Handle vector creation in HbaseRecordReader to avoid NullableInt vectors later

2017-10-20 Thread Prasad Nagaraj Subramanya (JIRA)
Prasad Nagaraj Subramanya created DRILL-5896: Summary: Handle vector creation in HbaseRecordReader to avoid NullableInt vectors later Key: DRILL-5896 URL:

[GitHub] drill issue #991: DRILL-5876: Remove netty-tcnative dependency from java-exe...

2017-10-20 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/991 @parthchandra open PR #1004 ---

[GitHub] drill pull request #1004: DRILL-5876: Use openssl profile to include netty-t...

2017-10-20 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/drill/pull/1004 DRILL-5876: Use openssl profile to include netty-tcnative dependency with the platform specific classifier You can merge this pull request into a Git repository by running: $ git pull

[GitHub] drill issue #991: DRILL-5876: Remove netty-tcnative dependency from java-exe...

2017-10-20 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/991 @vrozov your proposed solution is better and works for intellij and eclipse. Closing this PR and will wait for you to submit a new PR. Thanks!! ---

[GitHub] drill pull request #991: DRILL-5876: Remove netty-tcnative dependency from j...

2017-10-20 Thread parthchandra
Github user parthchandra closed the pull request at: https://github.com/apache/drill/pull/991 ---

[GitHub] drill issue #1002: DRILL-5888: Remove dependency of SSLConfig on hadoop.secu...

2017-10-20 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/1002 Yes, I will remove it. Thanks ! ---

[jira] [Resolved] (DRILL-5582) [Threat Modeling] Drillbit may be spoofed by an attacker and this may lead to data being written to the attacker's target instead of Drillbit

2017-10-20 Thread Sorabh Hamirwasia (JIRA)
[ https://issues.apache.org/jira/browse/DRILL-5582?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sorabh Hamirwasia resolved DRILL-5582. -- Resolution: Fixed > [Threat Modeling] Drillbit may be spoofed by an attacker and this

[jira] [Created] (DRILL-5895) Fix unit tests for mongo storage plugin

2017-10-20 Thread Volodymyr Tkach (JIRA)
Volodymyr Tkach created DRILL-5895: -- Summary: Fix unit tests for mongo storage plugin Key: DRILL-5895 URL: https://issues.apache.org/jira/browse/DRILL-5895 Project: Apache Drill Issue Type:

Re: [DISCUSS] Drill 1.12.0 release

2017-10-20 Thread Arina Yelchiyeva
Current status: Targeted for 1.12 release: DRILL-5832: Migrate OperatorFixture to use SystemOptionManager rather than mock (developer - Paul, code reviewer - ?) - waiting for the code review DRILL-5842: Refactor and simplify the fragment, operator contexts for testing (developer - Paul, code

[GitHub] drill issue #921: DRILL-4286 Graceful shutdown of drillbit

2017-10-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/921 @dvjyothsna is there any update on the status for this PR? Should it be targeted for the 1.12.0 release? ---

[GitHub] drill issue #1002: DRILL-5888: Remove dependency of SSLConfig on hadoop.secu...

2017-10-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/1002 One more thing, I have noticed that `TestSSLConfig#testInvalidHadoopKeystore()` contains the following line ``` String hadoopSSLFileProp = MessageFormat

[GitHub] drill issue #1003: DRILL-5893: Reverted the number of forked test processes ...

2017-10-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/1003 +1, LGTM. ---

Re: Excessive review comments

2017-10-20 Thread Arina Yelchiyeva
Ideally, there should be an option if you want to receive all notifications from github or not. All these notifications don't really bother me. As Parth mentioned this is the way to track dev activity. Subscribing to the pull request maybe not always be the best solution, since sometimes the