deniskuzZ commented on pull request #2503: URL: https://github.com/apache/hive/pull/2503#issuecomment-887407196
> Just checking – the goal of this change is to prevent concurrent compaction and DDL operations? > > As discussed offline, verifyTableOrPartitionIdHasNotChanged might not be needed because if the table (or partition) was dropped, then the compaction has been removed from the queue and can't be marked failed, right? Or is it needed after all? it's started as HIVE-24235 , but once we introduced read locks that's no longer needed. I'll remove it, however, that would require some tests changes that mocked that method. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
