----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55715/#review162281 -----------------------------------------------------------
lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTablePruneCause.java (line 151) <https://reviews.apache.org/r/55715/#comment233618> Should we say "Data for the requested metrics is only partially available. Partially available metrics are : %s. Please try again later." lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTablePruneCause.java (line 154) <https://reviews.apache.org/r/55715/#comment233619> Are we logging the old more verbose message( metric - time - %completion) somewhere ? It would be required for debugging the issue or to answer user queries when they pour in . - Puneet Gupta On Jan. 19, 2017, 12:25 p.m., Narayan Periwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55715/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2017, 12:25 p.m.) > > > Review request for lens. > > > Bugs: LENS-1384 > https://issues.apache.org/jira/browse/LENS-1384 > > > Repository: lens > > > Description > ------- > > We can make the error message for the data completeness check less verbose. > New message can be something like - Requested metrics are only partially > complete. Partially complete metrics: <comma separated list of metrics>. Try > again later or remove metric and re-run report. > > > Diffs > ----- > > > lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTablePruneCause.java > 2ad6e20 > lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java > c9e7c29 > > Diff: https://reviews.apache.org/r/55715/diff/ > > > Testing > ------- > > > Thanks, > > Narayan Periwal > >
