----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75235/#review226993 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On Oct. 23, 2024, 11:54 a.m., Rakesh Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/75235/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2024, 11:54 a.m.) > > > Review request for ranger, Dineshkumar Yadav, Kishor Gollapalliwar, Madhan > Neethiraj, Mehul Parikh, and sanket shelar. > > > Bugs: RANGER-4964 > https://issues.apache.org/jira/browse/RANGER-4964 > > > Repository: ranger > > > Description > ------- > > The issue occurred because the previous patch(RANGER-4809) failed to verify > if the x_trx_log table existed before modifying the x_trx_log_IDX_trx_id > index, which could result in errors. A table existence check needs to be > added to the patch. > > > Diffs > ----- > > security-admin/db/mysql/patches/074-create-index-for-x_trx_log.sql > 9a8f3c0d2 > security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql > 551660ab4 > security-admin/db/oracle/patches/067-create-x_trx_log_v2.sql > security-admin/db/oracle/patches/068-create-index-for-x_trx_log.sql > 1ef726b53 > security-admin/db/postgres/patches/074-create-index-for-x_trx_log.sql > 34cf77ccc > > security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql > 792fffd8d > security-admin/db/sqlanywhere/patches/067-create-x_trx_log_v2.sql > security-admin/db/sqlanywhere/patches/068-create-index-for-x_trx_log.sql > ae76a6de9 > security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql > eccf7f46f > security-admin/db/sqlserver/patches/067-create-x_trx_log_v2.sql > security-admin/db/sqlserver/patches/068-create-index-for-x_trx_log.sql > 3efb9eebb > > > Diff: https://reviews.apache.org/r/75235/diff/3/ > > > Testing > ------- > > Verified that the patch correctly checks for the existence of the x_trx_log > table before attempting to modify the x_trx_log_IDX_trx_id index. It executes > successfully without errors, even when the table does not exist. > > > Thanks, > > Rakesh Gupta > >
