[GitHub] hive pull request #154: Avoiding leaked tcp link to metastore

2017-02-27 Thread gyisgood
GitHub user gyisgood opened a pull request: https://github.com/apache/hive/pull/154 Avoiding leaked tcp link to metastore Using Hive.getWithFastCheck in Task.getHive will cause leaked tcp link to metastore when executing SQL You can merge this pull request into a Git repository

Re: [VOTE] Drop support for Java7 in master branch

2017-02-27 Thread Rajat Khandelwal
+1 On Tue, Feb 28, 2017 at 10:37 AM, Prasanth Jayachandran < pjayachand...@hortonworks.com> wrote: > +1 > > Thanks > Prasanth > > > > > On Mon, Feb 27, 2017 at 8:55 PM -0800, "Thejas Nair" < > thejas.n...@gmail.com> wrote: > > > There was a [DISCUSS] thread on the to

[jira] [Created] (HIVE-16055) HiveServer2: Prefetch a batch from the result file so that the RPC fetch request has results available in memory

2017-02-27 Thread Vaibhav Gumashta (JIRA)
Vaibhav Gumashta created HIVE-16055: --- Summary: HiveServer2: Prefetch a batch from the result file so that the RPC fetch request has results available in memory Key: HIVE-16055 URL: https://issues.apache.org/jira

[jira] [Created] (HIVE-16054) AMReporter should use application token instead of ugi.getCurrentUser

2017-02-27 Thread Prasanth Jayachandran (JIRA)
Prasanth Jayachandran created HIVE-16054: Summary: AMReporter should use application token instead of ugi.getCurrentUser Key: HIVE-16054 URL: https://issues.apache.org/jira/browse/HIVE-16054 P

Re: [VOTE] Drop support for Java7 in master branch

2017-02-27 Thread Prasanth Jayachandran
+1 Thanks Prasanth On Mon, Feb 27, 2017 at 8:55 PM -0800, "Thejas Nair" mailto:thejas.n...@gmail.com>> wrote: There was a [DISCUSS] thread on the topic of moving to jdk8 for unit tests [1], and many people also expressed the opinion that we should drop JDK 7 support in Hive. Public updates

[VOTE] Drop support for Java7 in master branch

2017-02-27 Thread Thejas Nair
There was a [DISCUSS] thread on the topic of moving to jdk8 for unit tests [1], and many people also expressed the opinion that we should drop JDK 7 support in Hive. Public updates by Oracle was stopped on Apr 2015 [2]. This vote thread proposes to dropping JDK 7 support in the next Apache Hive 2.

Re: Review Request 56687: Intern strings in various critical places to reduce memory consumption.

2017-02-27 Thread Rui Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56687/#review166991 --- ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java (line 3

[jira] [Created] (HIVE-16053) Remove newRatio from llap JAVA_OPTS_BASE

2017-02-27 Thread Siddharth Seth (JIRA)
Siddharth Seth created HIVE-16053: - Summary: Remove newRatio from llap JAVA_OPTS_BASE Key: HIVE-16053 URL: https://issues.apache.org/jira/browse/HIVE-16053 Project: Hive Issue Type: Bug

[jira] [Created] (HIVE-16052) MM tables: add exchange partition test after ACID integration

2017-02-27 Thread Sergey Shelukhin (JIRA)
Sergey Shelukhin created HIVE-16052: --- Summary: MM tables: add exchange partition test after ACID integration Key: HIVE-16052 URL: https://issues.apache.org/jira/browse/HIVE-16052 Project: Hive

[jira] [Created] (HIVE-16051) MM tables: skewjoin test fails

2017-02-27 Thread Sergey Shelukhin (JIRA)
Sergey Shelukhin created HIVE-16051: --- Summary: MM tables: skewjoin test fails Key: HIVE-16051 URL: https://issues.apache.org/jira/browse/HIVE-16051 Project: Hive Issue Type: Sub-task

Re: Review Request 56334: HIVE-12767: Implement table property to address Parquet int96 timestamp bug

2017-02-27 Thread Sergio Pena
> On Feb. 27, 2017, 11:20 p.m., Sergio Pena wrote: > > Ship It! Thanks Zsombor. This is good work. - Sergio --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56334/#review166963 ---

Re: Review Request 56334: HIVE-12767: Implement table property to address Parquet int96 timestamp bug

2017-02-27 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56334/#review166963 --- Ship it! Ship It! - Sergio Pena On Feb. 24, 2017, 2:56 p.m.,

Re: Review Request 52978: HIVE-14459 TestBeeLineDriver - migration and re-enable

2017-02-27 Thread Vihang Karajgaonkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52978/#review166945 --- LGTM (non-binding). Just one comment below. Thanks Peter! itests

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Vihang Karajgaonkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56995/ --- (Updated Feb. 27, 2017, 8:41 p.m.) Review request for hive, Aihua Xu, Ashutosh

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Vihang Karajgaonkar
> On Feb. 27, 2017, 6:01 p.m., Sahil Takiar wrote: > > ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveMetaStoreChecker.java, > > line 452 > > > > > > Can't the variable declartion be `Queue`, same for the other

Review Request 57102: HIVE-16040 union column expansion should take aliases from the leftmost branch

2017-02-27 Thread Sergey Shelukhin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57102/ --- Review request for hive, Ashutosh Chauhan and pengcheng xiong. Repository: hive

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Sahil Takiar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56995/#review166935 --- Ship it! LGTM besides one pending comment. - Sahil Takiar On

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Sahil Takiar
> On Feb. 27, 2017, 6:01 p.m., Sahil Takiar wrote: > > ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveMetaStoreChecker.java, > > lines 550-556 > > > > > > Not sure I follow this logic, why are two queues necessa

Re: Review Request 56687: Intern strings in various critical places to reduce memory consumption.

2017-02-27 Thread Misha Dmitriev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56687/ --- (Updated Feb. 27, 2017, 7:42 p.m.) Review request for hive, Chaoyu Tang, Mohit

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Sahil Takiar
> On Feb. 27, 2017, 6:01 p.m., Sahil Takiar wrote: > > ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveMetaStoreChecker.java, > > line 452 > > > > > > Can't the variable declartion be `Queue`, same for the other

Re: Review Request 56687: Intern strings in various critical places to reduce memory consumption.

2017-02-27 Thread Misha Dmitriev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56687/#review166916 --- ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMer

[jira] [Created] (HIVE-16050) Regression: Union of null with non-null

2017-02-27 Thread Gopal V (JIRA)
Gopal V created HIVE-16050: -- Summary: Regression: Union of null with non-null Key: HIVE-16050 URL: https://issues.apache.org/jira/browse/HIVE-16050 Project: Hive Issue Type: Bug Component

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Vihang Karajgaonkar
> On Feb. 27, 2017, 6:01 p.m., Sahil Takiar wrote: > > ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveMetaStoreChecker.java, > > line 452 > > > > > > Can't the variable declartion be `Queue`, same for the other

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Vihang Karajgaonkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56995/ --- (Updated Feb. 27, 2017, 6:44 p.m.) Review request for hive, Aihua Xu, Ashutosh

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Vihang Karajgaonkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56995/#review166900 --- ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveMetaStoreCheck

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Sahil Takiar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56995/#review166898 --- ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveMetaStoreCheck

[jira] [Created] (HIVE-16049) upgrade to jetty 9

2017-02-27 Thread Sean Busbey (JIRA)
Sean Busbey created HIVE-16049: -- Summary: upgrade to jetty 9 Key: HIVE-16049 URL: https://issues.apache.org/jira/browse/HIVE-16049 Project: Hive Issue Type: Improvement Reporter: Sea

Re: Review Request 56995: HIVE-15879 : Fix HiveMetaStoreChecker.checkPartitionDirs method

2017-02-27 Thread Peter Vary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56995/#review166889 --- Ship it! Me, and Yetus are satisfied :) Thanks for the patch!

Review Request 57094: HIVE-15996

2017-02-27 Thread Jesús Camacho Rodríguez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57094/ --- Review request for hive and Ashutosh Chauhan. Bugs: HIVE-15996 https://issu

[jira] [Created] (HIVE-16048) Hive UDF doesn't get the right evaluate method

2017-02-27 Thread Liao, Xiaoge (JIRA)
Liao, Xiaoge created HIVE-16048: --- Summary: Hive UDF doesn't get the right evaluate method Key: HIVE-16048 URL: https://issues.apache.org/jira/browse/HIVE-16048 Project: Hive Issue Type: Bug

Re: Review Request 56687: Intern strings in various critical places to reduce memory consumption.

2017-02-27 Thread Rui Li
> On Feb. 24, 2017, 7:38 a.m., Rui Li wrote: > > ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java, > > line 322 > > > > > > will this cause the hash map to resize since the default load

[jira] [Created] (HIVE-16047) Shouldn't try to get KeyProvider unless encryption is enabled

2017-02-27 Thread Rui Li (JIRA)
Rui Li created HIVE-16047: - Summary: Shouldn't try to get KeyProvider unless encryption is enabled Key: HIVE-16047 URL: https://issues.apache.org/jira/browse/HIVE-16047 Project: Hive Issue Type: Bug

[jira] [Created] (HIVE-16046) Broadcasting small table for Hive on Spark

2017-02-27 Thread liyunzhang_intel (JIRA)
liyunzhang_intel created HIVE-16046: --- Summary: Broadcasting small table for Hive on Spark Key: HIVE-16046 URL: https://issues.apache.org/jira/browse/HIVE-16046 Project: Hive Issue Type: Bug