-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72291/#review220153
-----------------------------------------------------------


Ship it!




Just a question, maybe not relevant.

Thanks for the patch LGTM +1 (pending tests :))


standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/CompactionTxnHandler.java
Lines 545-560 (original), 496-512 (patched)
<https://reviews.apache.org/r/72291/#comment308477>

    How confident we are with this query?
    Maybe delete immediately, like:
    DELETE FROM TXN_TO_WRITE_ID WHERE T2W_TXNID < (SELECT MIN...)
    The hard part is that it is hard to log what happened


- Peter Vary


On márc. 31, 2020, 1:51 du, Laszlo Pinter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72291/
> -----------------------------------------------------------
> 
> (Updated márc. 31, 2020, 1:51 du)
> 
> 
> Review request for hive and Peter Vary.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-23107: Remove MIN_HISTORY_LEVEL table
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Cleaner.java 
> 54b616e60c73fa1005c6d679ea76d65e01a0749d 
>   
> standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/CompactionInfo.java
>  bf91ae704c83722502acaf445061bf297fde6a6f 
>   
> standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/CompactionTxnHandler.java
>  19a95b64db64b7f8bf8e82fbdedf5b54cd30aed3 
>   
> standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnDbUtil.java
>  ef88240a791cbc553b6150b54701c7df8daf3b49 
>   
> standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java
>  74ef88545e6fd680b311ca5c5e4d87f053af1026 
>   
> standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnStore.java
>  41d2e7924b1286f82d212e051714107505fe9661 
>   
> standalone-metastore/metastore-server/src/main/sql/derby/hive-schema-4.0.0.derby.sql
>  48ad67623143fcba2bb2507cc36ad46a58c13b75 
>   
> standalone-metastore/metastore-server/src/main/sql/derby/upgrade-3.2.0-to-4.0.0.derby.sql
>  7a230bde3a4dcd2504538bf8122db0bb2a59932f 
>   
> standalone-metastore/metastore-server/src/main/sql/mssql/hive-schema-4.0.0.mssql.sql
>  9ed7f4f8192e1613f8942af3e9496c3ef7f1f04f 
>   
> standalone-metastore/metastore-server/src/main/sql/mssql/upgrade-3.2.0-to-4.0.0.mssql.sql
>  12d24e9a569310c57d86b1229deaa2b1d080e0b8 
>   
> standalone-metastore/metastore-server/src/main/sql/mysql/hive-schema-4.0.0.mysql.sql
>  bc34b511a978af2a6704e47d8f73c9604c6cebf9 
>   
> standalone-metastore/metastore-server/src/main/sql/mysql/upgrade-3.2.0-to-4.0.0.mysql.sql
>  13f03bce6d579c18ff9a14e09133063fdcdbf7af 
>   
> standalone-metastore/metastore-server/src/main/sql/oracle/hive-schema-4.0.0.oracle.sql
>  8482b5942c6284eda63ec7d2aa7c1202abb3db49 
>   
> standalone-metastore/metastore-server/src/main/sql/oracle/upgrade-3.2.0-to-4.0.0.oracle.sql
>  cbfdd861fd545ce786f5e267b284196f5fd9af03 
>   
> standalone-metastore/metastore-server/src/main/sql/postgres/hive-schema-4.0.0.postgres.sql
>  aa35a7a7b392d4d46480c03ed95ffdec12b03b22 
>   
> standalone-metastore/metastore-server/src/main/sql/postgres/upgrade-3.2.0-to-4.0.0.postgres.sql
>  9462328a5f0c6dda8cffb683860153b8eb3aacec 
> 
> 
> Diff: https://reviews.apache.org/r/72291/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Laszlo Pinter
> 
>

Reply via email to