----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67899/ -----------------------------------------------------------
Review request for sentry, kalyan kumar kalvagadda and Sergio Pena. Bugs: sentry-2299 https://issues.apache.org/jira/browse/sentry-2299 Repository: sentry Description ------- Sentry client at NameNode does not create entry for external table. When user rename an external table with different path, null exception happens. The fix is to check if old entry exists or not. If exists, move old paths to new table. Otherwise, skip that step. Diffs ----- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java 3919d60 sentry-hdfs/sentry-hdfs-common/src/test/java/org/apache/sentry/hdfs/TestHMSPaths.java 20ed97c Diff: https://reviews.apache.org/r/67899/diff/1/ Testing ------- Add new test case for renaming external table with different paths. It passes Thanks, Na Li