> On May 4, 2020, 3 p.m., Denys Kuzmenko wrote: > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnUtils.java > > Lines 400 (patched) > > <https://reviews.apache.org/r/72465/diff/2/?file=2230140#file2230140line400> > > > > Is is a generic txn object (i.e TxnInfo)? I don't think TxnUtils is a > > good place for domain objects. > > Peter Varga wrote: > I didn't really know which way to go. The TxnInfo is missing the type > field and I did not want to change the api. > The tables used by txnhandler don't have domain objects, should I create > one in the model package for txns? I think that would be confusing too, since > it would seem like the txns table is managed by jdo. > What do you suggest?
I would create OpenTxnList under org.apache.hadoop.hive.metastore.txn (similarily to CompactionInfo) - same as you did with extra 2 methods: - toOpenTxns (OpenTxnsResponse) - toOpenTxnsInfo (OpenTxnsInfoResponse) and place dto conversion logic there - Denys ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72465/#review220593 ----------------------------------------------------------- On May 4, 2020, 1:22 p.m., Peter Varga wrote: > > ----------------------------------------------------------- > 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. > > > Repository: hive-git > > > Description > ------- > > * Merge getOpenTxns and getOpenTxnInfo to avoid code duplication > * Remove TxnStatus character constants and use the enum values > > > Diffs > ----- > > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/CompactionTxnHandler.java > a1bc10955a > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java > 8fded608d0 > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnUtils.java > 4ee1a45aae > > > Diff: https://reviews.apache.org/r/72465/diff/2/ > > > Testing > ------- > > > Thanks, > > Peter Varga > >
