----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50133/#review143075 -----------------------------------------------------------
Fix it, then Ship it! lens-server/src/main/java/org/apache/lens/server/ui/QueryServiceUIResource.java (line 118) <https://reviews.apache.org/r/50133/#comment208783> Might conflict with other patch - Amareshwari Sriramadasu On July 21, 2016, 8:43 a.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50133/ > ----------------------------------------------------------- > > (Updated July 21, 2016, 8:43 a.m.) > > > Review request for lens. > > > Bugs: LENS-996 > https://issues.apache.org/jira/browse/LENS-996 > > > Repository: lens > > > Description > ------- > > 1. fromDate and endDate should also take human readable dates. > DateUtil.resolveDate can be used. > 2. the status filter should be case-insensitive. > 3. (maybe) the user filter and status filter can take multiple comma > separated values and return the union in the result. > > > Diffs > ----- > > lens-cube/src/main/java/org/apache/lens/cube/metadata/DateUtil.java > b82cd95b5633b91e03cf1df8ac2602cb7798099a > > lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryExecutionService.java > d10ad099b7220545802a3fcc7fd6e32a1c5b272a > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > be0e2c80a6c0081537feaea330192dc85949ad05 > > lens-server/src/main/java/org/apache/lens/server/query/QueryServiceResource.java > 803bbe3db982dc74ba2a1bdba94fc68879dcd351 > > lens-server/src/main/java/org/apache/lens/server/ui/QueryServiceUIResource.java > 304dc8efad8bd27c88b58b6ca319dfedcd4ac239 > > Diff: https://reviews.apache.org/r/50133/diff/ > > > Testing > ------- > > > Thanks, > > Rajat Khandelwal > >
