> On Aug. 10, 2018, 10:24 p.m., Na Li wrote: > > have you verified on Postgres that upgrade from for DB on '1.5.0-cdh5' > > version, and then run script to upgrade to '1.5.0-cdh5-2' with your change > > to see upgrade successful?
I ran the upgrade script on 1.5.0-cdh5 environment and it succesfully added the index. Then I ran the upgrade script again and it logged that the index already exists - Arjun ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68295/#review207094 ----------------------------------------------------------- On Aug. 10, 2018, 5:20 p.m., Arjun Mishra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68295/ > ----------------------------------------------------------- > > (Updated Aug. 10, 2018, 5:20 p.m.) > > > Review request for sentry, kalyan kumar kalvagadda, Na Li, and Sergio Pena. > > > Bugs: SENTRY-2333 > https://issues.apache.org/jira/browse/SENTRY-2333 > > > Repository: sentry > > > Description > ------- > > Create index AUTHZ_PATH_FK_IDX at table AUTHZ_PATH for Postgres only when it > does not exist > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.postgres.sql > cdc2ff7f2 > > > Diff: https://reviews.apache.org/r/68295/diff/5/ > > > Testing > ------- > > > Thanks, > > Arjun Mishra > >
