I think you are right - there is no reason to do that. I am not sure whether we can actually have changed DB name in ALTER TABLE event at all.
On Tue, Apr 10, 2018 at 12:58 PM, Na Li <lina...@cloudera.com> wrote: > Sasha, > > > In FullUpdateModifier.alterTable(), why did you have the section that > "Walk through all tables and rename DB part of the AUTH name"? > > This is alter table processing, so we should only process one table, not > all tables in a database. Also, in NotificationProcessor, we don't change > all tables in a database if the database name changes. > > Can you review https://reviews.apache.org/r/65768/? I removed that > section. > > Thanks, > > Lina > > > > > > > >
