> On June 28, 2017, 2:10 p.m., Dmytro Grinenko wrote: > > ambari-server/src/test/java/org/apache/ambari/server/orm/DBAccessorImplTest.java > > Lines 683 (patched) > > <https://reviews.apache.org/r/60498/diff/1/?file=1766136#file1766136line683> > > > > If we checking here for idempotent issue, does we need to check the > > resulset? > > > > In case if function would be not idempotent, second update call will > > cause exception, hence we need to check for this
Yes, it's not strictly necessary. Should I remove everything after the `moveColumnToAnotherTable` call then? - Attila ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60498/#review179091 ----------------------------------------------------------- On June 28, 2017, 10:59 a.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60498/ > ----------------------------------------------------------- > > (Updated June 28, 2017, 10:59 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Dmytro Grinenko, Dmitro > Lisnichenko, and Jonathan Hurley. > > > Bugs: AMBARI-21362 > https://issues.apache.org/jira/browse/AMBARI-21362 > > > Repository: ambari > > > Description > ------- > > Check for source column existence, too. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/orm/DBAccessorImpl.java > 13e7d7d148795325cd0d1c698eae084f2f00a411 > > ambari-server/src/test/java/org/apache/ambari/server/orm/DBAccessorImplTest.java > ca2674c7109d8f2e4b75edbb85dbe5e05c8b1927 > > > Diff: https://reviews.apache.org/r/60498/diff/1/ > > > Testing > ------- > > Manually tested: > * real upgrade from 2.4.2 for any regression > * simulated retry: upgrade from 2.5.2 where metainfo says it's 2.5.0 > > New unit test. > > > Thanks, > > Attila Doroszlai > >
