[
https://issues.apache.org/jira/browse/HIVE-23560?focusedWorklogId=459782&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-459782
]
ASF GitHub Bot logged work on HIVE-23560:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 16/Jul/20 11:38
Start Date: 16/Jul/20 11:38
Worklog Time Spent: 10m
Work Description: pkumarsinha commented on a change in pull request #1232:
URL: https://github.com/apache/hive/pull/1232#discussion_r455721742
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/lockmgr/HiveTxnManager.java
##########
@@ -186,6 +186,18 @@ void replTableWriteIdState(String validWriteIdList, String
dbName, String tableN
*/
ValidTxnList getValidTxns() throws LockException;
+ /**
+ * Get the transactions that are currently valid. The resulting
+ * {@link ValidTxnList} object can be passed as string to the processing
+ * tasks for use in the reading the data. This call should be made once up
+ * front by the planner and should never be called on the backend,
+ * as this will violate the isolation level semantics.
+ * @return list of valid transactions.
+ * @param txnTypes list of transaction types that should be excluded.
+ * @throws LockException
+ */
+ ValidTxnList getValidTxns(List<TxnType> txnTypes) throws LockException;
Review comment:
Currently txnTypes is set of 5 enum values, but that could change. Also,
imagine if someone wants to get a single type, he will have to pass n-1 type in
exclude list. Since it is exposed as HMS api, it could be used outside of
replication cases as well. Since these are public interfaces, it is important.
with this interface in place, if someone wants to add additional api to get txn
with matching types, how will that extension look like?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 459782)
Time Spent: 2h 10m (was: 2h)
> Optimize bootstrap dump to abort only write Transactions
> --------------------------------------------------------
>
> Key: HIVE-23560
> URL: https://issues.apache.org/jira/browse/HIVE-23560
> Project: Hive
> Issue Type: Task
> Reporter: Aasha Medhi
> Assignee: Aasha Medhi
> Priority: Major
> Labels: pull-request-available
> Attachments: HIVE-23560.01.patch, HIVE-23560.02.patch,
> HIVE-23560.03.patch, Optimize bootstrap dump to avoid aborting all
> transactions.pdf
>
> Time Spent: 2h 10m
> Remaining Estimate: 0h
>
> Currently before doing a bootstrap dump, we abort all open transactions after
> waiting for a configured time. We are proposing to abort only write
> transactions for the db under replication and leave the read and repl created
> transactions as is.
> This doc attached talks about it in detail
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
