-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73404/
-----------------------------------------------------------
(Updated June 4, 2021, 8:43 p.m.)
Review request for atlas, Mandar Ambawane, Radhika Kundam, and Sarath
Subramanian.
Changes
-------
Updates include: Addressed review comments.
Bugs: ATLAS-4324
https://issues.apache.org/jira/browse/ATLAS-4324
Repository: atlas
Description
-------
**Approach**
Updates:
- Simplified approach.
- Maintained abstraction.
- Updated unit tests.
Diffs (updated)
-----
addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java
94ef22577
addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/utils/ActiveEntityFilter.java
737c637bf
addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/utils/HiveDDLEntityFilter.java
9163c4709
addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/utils/ActiveEntityFilterTest.java
4dde1dc2f
addons/hive-bridge/src/test/resources/json/hs2-create-db-v2.json 881ee105e
addons/hive-bridge/src/test/resources/json/hs2-create-db-with-no-pathentities-to-retain-v2.json
42553b5cf
addons/hive-bridge/src/test/resources/json/hs2-create-db-with-no-pathentities-to-retain.json
a5b810f72
addons/hive-bridge/src/test/resources/json/hs2-create-process-v2.json
8e55b7282
addons/hive-bridge/src/test/resources/json/hs2-create-table-v2.json 801918e04
addons/hive-bridge/src/test/resources/json/hs2-load-inpath-v2.json dd31aa073
addons/hive-bridge/src/test/resources/json/hs2-load-inpath-with-no-pathentities-to-retain-v2.json
7f90d1986
addons/hive-bridge/src/test/resources/json/hs2-load-inpath-with-no-pathentities-to-retain.json
dd31aa073
addons/hive-bridge/src/test/resources/json/hs2-load-inpath.json dd31aa073
addons/hive-bridge/src/test/resources/json/hs2-table-rename-v2.json f2b2bf7a8
Diff: https://reviews.apache.org/r/73404/diff/2/
Changes: https://reviews.apache.org/r/73404/diff/1-2/
Testing (updated)
-------
**Unit tests**
- Additional unit test.
**PC Build**
https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/617/
Thanks,
Ashutosh Mestry
