[jira] [Created] (HIVE-23361) Optimising privilege synchroniser

Simhadri G created HIVE-23361: - Summary: Optimising privilege synchroniser Key: HIVE-23361 URL: https://issues.apache.org/jira/browse/HIVE-23361 Project: Hive Issue Type: Improvement Co

Re: Review Request 72431: HIVE-23206

> On April 27, 2020, 6:02 p.m., Vineet Garg wrote: > > ql/src/test/results/clientpositive/llap/keep_uniform.q.out > > Lines 946 (patched) > > > > > > Why is there an extra join in the plan now? > > Krisztian Kasa wr

Review Request 72465: HIVE-23340 TxnHandler cleanup

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72465/ --- Review request for hive and Denys Kuzmenko. Repository: hive-git Description

Re: Review Request 72431: HIVE-23206

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72431/ --- (Updated May 4, 2020, 7:55 a.m.) Review request for hive, Jesús Camacho Rodrígu

Re: Review Request 72451: Add tests to cover database managed location related DDL and fix minor issues

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72451/#review220565 --- ql/src/java/org/apache/hadoop/hive/ql/ddl/table/create/show/ShowC

What's master key in hive metastore?

Dear all, I find several APIs about master key in hive metastore and IMetastoreClient. I searched around but didn't find any usage in hive DDL tasks. Any idea where this master key is designed for? Thanks in advance, bargitta int addMasterKey(String key) throws MetaException, TException; void u

[jira] [Created] (HIVE-23362) Repl dump returns dump location and repl id even in case of failure

Aasha Medhi created HIVE-23362: -- Summary: Repl dump returns dump location and repl id even in case of failure Key: HIVE-23362 URL: https://issues.apache.org/jira/browse/HIVE-23362 Project: Hive

Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72466/ --- Review request for hive, Jesús Camacho Rodríguez, John Sherman, Zoltan Haindrich

Review Request 72469: HIVE-23325: Clean up cleanup tasks for TxnHandler/CompactionTxnHandler

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72469/ --- Review request for hive, Denys Kuzmenko and Peter Vary. Repository: hive-git

Re: Review Request 72465: HIVE-23340 TxnHandler cleanup

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72465/ --- (Updated May 4, 2020, 1:22 p.m.) Review request for hive and Denys Kuzmenko.

Review Request 72470: ACID: Concurrent MERGE INSERT operations produce duplicates

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72470/ --- Review request for hive, Marton Bod, Peter Varga, and Peter Vary. Bugs: HIVE-23

Re: Review Request 72470: ACID: Concurrent MERGE INSERT operations produce duplicates

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72470/ --- (Updated May 4, 2020, 1:32 p.m.) Review request for hive, Marton Bod, Peter Var

[jira] [Created] (HIVE-23363) Upgrade DataNucleus dependency to 5.2

Zoltan Chovan created HIVE-23363: Summary: Upgrade DataNucleus dependency to 5.2 Key: HIVE-23363 URL: https://issues.apache.org/jira/browse/HIVE-23363 Project: Hive Issue Type: Improvement

Re: Review Request 72465: HIVE-23340 TxnHandler cleanup

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72465/#review220593 --- standalone-metastore/metastore-server/src/main/java/org/apache/ha

Re: Review Request 72470: ACID: Concurrent MERGE INSERT operations produce duplicates

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72470/#review220595 --- Looks good to me, just a couple of questions. ql/src/test/org/ap

Re: Review Request 72469: HIVE-23325: Clean up cleanup tasks for TxnHandler/CompactionTxnHandler

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72469/#review220594 --- LGTM , just few questions ql/src/java/org/apache/hadoop/hive/ql/

Re: Review Request 72469: HIVE-23325: Clean up cleanup tasks for TxnHandler/CompactionTxnHandler

> On May 4, 2020, 3:21 p.m., Denys Kuzmenko wrote: > > LGTM , just few questions Thanks for the review! > On May 4, 2020, 3:21 p.m., Denys Kuzmenko wrote: > > ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Initiator.java > > Line 153 (original) > >

Re: Review Request 72469: HIVE-23325: Clean up cleanup tasks for TxnHandler/CompactionTxnHandler

> On May 4, 2020, 3:21 p.m., Denys Kuzmenko wrote: > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/AcidCompactionHistoryService.java > > Line 31 (original), 32 (patched) > > >

[jira] [Created] (HIVE-23364) Push down null-safe cast-s

Zoltan Haindrich created HIVE-23364: --- Summary: Push down null-safe cast-s Key: HIVE-23364 URL: https://issues.apache.org/jira/browse/HIVE-23364 Project: Hive Issue Type: Improvement

Re: Review Request 72470: ACID: Concurrent MERGE INSERT operations produce duplicates

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72470/ --- (Updated May 4, 2020, 4:33 p.m.) Review request for hive, Marton Bod, Peter Var

Re: Review Request 72470: ACID: Concurrent MERGE INSERT operations produce duplicates

> On May 4, 2020, 3:21 p.m., Marton Bod wrote: > > ql/src/test/org/apache/hadoop/hive/ql/lockmgr/TestDbTxnManager2.java > > Lines 2231 (patched) > > > > > > nit: this appears in a few places, might make sense to ext

Re: Review Request 72470: ACID: Concurrent MERGE INSERT operations produce duplicates

> On May 4, 2020, 3:21 p.m., Marton Bod wrote: > > Looks good to me, just a couple of questions. Thank you for the review! > On May 4, 2020, 3:21 p.m., Marton Bod wrote: > > ql/src/test/org/apache/hadoop/hive/ql/lockmgr/TestDbTxnManager2.java > > Lines 2226 (patched) > >

Re: What's master key in hive metastore?

It is used in storing delegation tokens, which HS2 and the megastore need to act on the user's behalf when reading files, launching jobs, etc. See DelegationTokenStore. Alan. On Mon, May 4, 2020 at 2:00 AM Xi Chen wrote: > Dear all, > > I find several APIs about master key in hive metastore an

[jira] [Created] (HIVE-23365) Put RS deduplication optimization under cost based decision

Jesus Camacho Rodriguez created HIVE-23365: -- Summary: Put RS deduplication optimization under cost based decision Key: HIVE-23365 URL: https://issues.apache.org/jira/browse/HIVE-23365 Project

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72466/#review220597 --- ql/src/test/results/clientpositive/llap/join45.q.out Line 1104 (o

[jira] [Created] (HIVE-23366) Investigate why bucket_map_join*, bucketcontext_* and bucketmapjoin* tests are missing bucket map join when run under TestMiniLlapLocalCliDriver

Vineet Garg created HIVE-23366: -- Summary: Investigate why bucket_map_join*, bucketcontext_* and bucketmapjoin* tests are missing bucket map join when run under TestMiniLlapLocalCliDriver Key: HIVE-23366 URL: https:/

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72466/#review220607 --- ql/src/test/results/clientpositive/llap/groupby_duplicate_key.q.o

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72466/#review220608 --- ql/src/test/results/clientpositive/llap/groupby_sort_8.q.out Line

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

> On May 4, 2020, 5:20 p.m., Zoltan Haindrich wrote: > > ql/src/test/results/clientpositive/llap/join_cond_pushdown_unqual5.q.out > > Line 100 (original), 114 (patched) > > > > > > the CASE have been pushed into the

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

> On May 4, 2020, 9:20 p.m., Steve Carlin wrote: > > ql/src/test/results/clientpositive/llap/groupby_sort_skew_1_23.q.out > > Line 49 (original), 49 (patched) > > > > > > This file is a bit overwhelming to check. 8K

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72466/#review220610 --- ql/src/test/results/clientpositive/llap/input_part3.q.out Line 24

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72466/#review220611 --- ql/src/test/results/clientpositive/llap/infer_bucket_sort_groupin

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72466/#review220613 --- ql/src/test/results/clientpositive/llap/leadlag_queries.q.out Lin

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72466/#review220612 --- I didn't call out every spot where order changed of results unless

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

> On May 5, 2020, 1:59 a.m., John Sherman wrote: > > ql/src/test/results/clientpositive/llap/mapjoin_hook.q.out > > Line 41 (original), 41 (patched) > > > > > > Same as above Not same as above, prior one failed due t

Re: Review Request 72437: Reduce number of DB calls in ObjectStore::getPartitionsByExprInternal

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72437/#review220616 --- ql/src/java/org/apache/hadoop/hive/ql/metadata/SessionHiveMetaSto

[jira] [Created] (HIVE-23367) Missed filterExpr simplification in merge_join_1

Zoltan Haindrich created HIVE-23367: --- Summary: Missed filterExpr simplification in merge_join_1 Key: HIVE-23367 URL: https://issues.apache.org/jira/browse/HIVE-23367 Project: Hive Issue Typ

Re: Review Request 72466: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 2

> On May 5, 2020, 3:59 a.m., John Sherman wrote: > > ql/src/test/results/clientpositive/llap/mapjoin_hook.q.out > > Lines 56-61 (original) > > > > > > These used to be failures that then ran a backuptask BackupTask i