-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/75377/
-----------------------------------------------------------
(Updated May 8, 2025, 4:08 p.m.)
Review request for ranger, Madhan Neethiraj and Ramesh Mani.
Changes
-------
Updated the patch as per review comments
Bugs: RANGER-5170
https://issues.apache.org/jira/browse/RANGER-5170
Repository: ranger
Description
-------
Improve Ranger logging to capture all hierarchical events for better visibility
and traceability.
Currently, Ranger logs only direct changes to an entity, missing related
updates across interconnected entities. For example, when a resource is created
or updated, only a resource log is generated, making it difficult to track
associated changes in dependent entities.
With this change, Ranger will generate logs for all impacted entities in the
hierarchy. Example:
If a resource is modified, logs will be created for:
Resource
Datashare (if the resource is part of a datashare)
DataShareInDataset (if the datashare is linked to a dataset)
Dataset
This transaction will log all the hierarchical logs and it'll include details
of the changed resource to ensure complete visibility.
Diffs (updated)
-----
security-admin/src/main/java/org/apache/ranger/biz/AssetMgr.java ec39448f3
security-admin/src/main/java/org/apache/ranger/db/XXGdsDatasetDao.java
b0dbef3e8
security-admin/src/main/java/org/apache/ranger/db/XXGdsDatasetPolicyMapDao.java
b1ff36ad0
security-admin/src/main/java/org/apache/ranger/db/XXGdsProjectDao.java
a005519e7
security-admin/src/main/java/org/apache/ranger/db/XXGdsProjectPolicyMapDao.java
b3529e840
security-admin/src/main/java/org/apache/ranger/rest/AssetREST.java d29e44eff
security-admin/src/main/java/org/apache/ranger/service/RangerAuditedModelService.java
edf514b1f
security-admin/src/main/resources/META-INF/jpa_named_queries.xml 3559877d5
security-admin/src/test/java/org/apache/ranger/rest/TestAssetREST.java
7bddf1874
Diff: https://reviews.apache.org/r/75377/diff/4/
Changes: https://reviews.apache.org/r/75377/diff/3-4/
Testing
-------
Tested locally.Attached example scenario of logs before and after the change.
File Attachments (updated)
----------------
Example Logs with updated patch
https://reviews.apache.org/media/uploaded/files/2025/05/08/26a51489-1f07-4639-8ff2-4edd4d1dbba6__Examples_previous_vs_current_logging.png
Example Logs with updated patch
https://reviews.apache.org/media/uploaded/files/2025/05/08/bb02df21-20a7-46ad-a5bf-2a02c930156b__Examples_previous_vs_current_logging.png
Thanks,
Radhika Kundam
