Sasha, We can. And I added the test case in e2e tests, and it works.
On Tue, Apr 10, 2018 at 5:34 PM, Alexander Kolbasov <ak...@cloudera.com> wrote: > 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 >> >> >> >> >> >> >> >> >