> On Jan. 19, 2017, 2:06 p.m., Puneet Gupta wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTablePruneCause.java, > > line 154 > > <https://reviews.apache.org/r/55715/diff/1/?file=1608971#file1608971line154> > > > > 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 .
Yes, we are logging the entire verbose message that will help in debugging the issue, as you mentioned. - Narayan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55715/#review162281 ----------------------------------------------------------- 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 > >
