[
https://issues.apache.org/jira/browse/HIVE-13622?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15285939#comment-15285939
]
Eugene Koifman commented on HIVE-13622:
---------------------------------------
Test Results
{noformat}
Test Name Duration Age
org.apache.hadoop.hive.cli.TestHBaseCliDriver.testCliDriver_hbase_queries
5 min 32 sec 1
org.apache.hadoop.hive.llap.tez.TestLlapDaemonProtocolClientProxy.testSingleInvocationPerNode
5 sec 1
org.apache.hadoop.hive.llap.tezplugins.TestLlapTaskSchedulerService.testDelayedLocalityNodeCommErrorImmediateAllocation
10 sec 1
org.apache.hadoop.hive.metastore.TestMetaStoreMetrics.testConnections 87 ms
1
org.apache.hive.jdbc.TestSSL.testSSLConnectionWithURL 32 sec 1
org.apache.hadoop.hive.llap.daemon.impl.TestTaskExecutorService.testPreemptionQueueComparator
5 sec 1
org.apache.hadoop.hive.llap.daemon.impl.comparator.TestShortestJobFirstComparator.testWaitQueueComparatorWithinDagPriority
5.3 sec 4
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver_insert_values_orig_table_use_metadata
41 sec 6
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver_join32 8.4 sec 6
org.apache.hadoop.hive.cli.TestEncryptedHDFSCliDriver.testCliDriver_encryption_insert_partition_static
1 min 57 sec 7
org.apache.hadoop.hive.cli.TestEncryptedHDFSCliDriver.testCliDriver_encryption_insert_partition_dynamic
1 min 15 sec 7
org.apache.hive.minikdc.TestHiveAuthFactory.testStartTokenManagerForMemoryTokenStore
1.2 sec 7
org.apache.hive.minikdc.TestHiveAuthFactory.testStartTokenManagerForDBTokenStore
0.32 sec 7
org.apache.hive.minikdc.TestMiniHiveKdc.testLogin 1 min 49 sec 7
org.apache.hadoop.hive.llap.tez.TestConverters.testFragmentSpecToTaskSpec
54 ms 27
org.apache.hive.service.cli.session.TestHiveSessionImpl.testLeakOperationHandle
28 sec 27
org.apache.hadoop.hive.llap.tezplugins.TestLlapTaskCommunicator.testFinishableStateUpdateFailure
5.1 sec 40
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver_selectindate 13 sec
59
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver_avrocountemptytbl
9.6 sec 59
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver_order_null 46 sec
59
org.apache.hadoop.hive.cli.TestEncryptedHDFSCliDriver.testCliDriver_encryption_join_with_different_encryption_keys
1 min 36 sec 59
org.apache.hadoop.hive.cli.TestMiniSparkOnYarnCliDriver.testCliDriver_index_bitmap3
9.7 sec 59
org.apache.hadoop.hive.cli.TestMinimrCliDriver.org.apache.hadoop.hive.cli.TestMinimrCliDriver
1 min 16 sec 59
{noformat}
Failures are not related
> WriteSet tracking optimizations
> -------------------------------
>
> Key: HIVE-13622
> URL: https://issues.apache.org/jira/browse/HIVE-13622
> Project: Hive
> Issue Type: Bug
> Components: Transactions
> Affects Versions: 1.3.0, 2.1.0
> Reporter: Eugene Koifman
> Assignee: Eugene Koifman
> Priority: Critical
> Attachments: HIVE-13622.2.patch, HIVE-13622.3.patch,
> HIVE-13622.4.patch
>
>
> HIVE-13395 solves the the lost update problem with some inefficiencies.
> 1. TxhHandler.OperationType is currently derived from LockType. This doesn't
> distinguish between Update and Delete but would be useful. See comments in
> TxnHandler. Should be able to pass in Insert/Update/Delete info from client
> into TxnHandler.
> 2. TxnHandler.addDynamicPartitions() should know the OperationType as well
> from the client. It currently extrapolates it from TXN_COMPONENTS. This
> works but requires extra SQL statements and is thus less performant. It will
> not work multi-stmt txns. See comments in the code.
> 3. TxnHandler.checkLock() see more comments around
> "isPartOfDynamicPartitionInsert". If TxnHandler knew whether it is being
> called as part of an op running with dynamic partitions, it could be more
> efficient. In that case we don't have to write to TXN_COMPONENTS at all
> during lock acquisition. Conversely, if not running with DynPart then, we
> can kill current txn on lock grant rather than wait until commit time.
> 4. TxnHandler.addDynamicPartitions() - the insert stmt here should combing
> multiple rows into single SQL stmt (but with a limit for extreme cases)
> 5. TxnHandler.enqueueLockWithRetry() - this currently adds components that
> are only being read to TXN_COMPONENTS. This is useless at best since read
> op don't generate anything to compact. For example, delete from T where t1
> in (select c1 from C) - no reason to add C to txn_components but we do.
>
> All of these require some Thrift changes
> Once done, re-enable TestDbTxnHandler2.testWriteSetTracking11()
> Also see comments in
> [here|https://issues.apache.org/jira/browse/HIVE-13395?focusedCommentId=15271712&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15271712]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
