----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75360/#review227244 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/biz/GdsDBStore.java Lines 1801 (patched) <https://reviews.apache.org/r/75360/#comment315487> Please add debug statements for these new methods security-admin/src/main/java/org/apache/ranger/biz/GdsDBStore.java Lines 1849 (patched) <https://reviews.apache.org/r/75360/#comment315488> nit: Please have a space after "," as this may fail the verify pmd check. - Ramesh Mani On Feb. 10, 2025, 11:38 p.m., Radhika Kundam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/75360/ > ----------------------------------------------------------- > > (Updated Feb. 10, 2025, 11:38 p.m.) > > > Review request for ranger, Madhan Neethiraj and Ramesh Mani. > > > Bugs: RANGER-5129 > https://issues.apache.org/jira/browse/RANGER-5129 > > > Repository: ranger > > > Description > ------- > > Enhance the existing enhancedSummary API to support filtering datasets based > on their validity schedule and expiration date. The API should return > datasets that have a defined validity schedule and whose expiration date > falls within the specified date range. > > Filtering Criteria: > > The dataset must have a validity schedule. > The expiration date of the dataset should be within the provided range: > validityExpiryFrom – Start of the expiration date range > validityExpiryTo – End of the expiration date range > This enhancement ensures that only relevant datasets with scheduled validity > and expiring within the given timeframe are retrieved. > > API: curl GET <ranger-admin > url>/service/gds/dataset/enhancedsummary?validityExpiryFrom=<yyyy/MM/dd > HH:mm:ss>&validityExpiryTo=<yyyy/MM/dd HH:mm:ss> > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerValidityScheduleEvaluator.java > 027c2140c > agents-common/src/main/java/org/apache/ranger/plugin/util/SearchFilter.java > a260a8782 > security-admin/src/main/java/org/apache/ranger/biz/GdsDBStore.java > 312a67a28 > security-admin/src/main/java/org/apache/ranger/common/RangerSearchUtil.java > 6c93d7104 > security-admin/src/test/java/org/apache/ranger/rest/TestGdsREST.java > 68d62fc31 > > > Diff: https://reviews.apache.org/r/75360/diff/1/ > > > Testing > ------- > > Tested locally. > > > Thanks, > > Radhika Kundam > >