xuchuanyin commented on a change in pull request #3409: [CARBONDATA-3544] cli support summary statistics for all columns
URL:
https://github.com/apache/carbondata/pull/3409#discussion_r333837008
##########
File path: tools/cli/src/test/java/org/apache/carbondata/tool/CarbonCliTest.java
##########
@@ -260,6 +264,31 @@ public void testSummaryPageMeta() {
Assert.assertTrue(output.contains(expectedOutput));
}
+ @Test
+ public void testSummaryAllColumns() {
+ String[] args = { "-cmd", "summary", "-p", path, "-C" };
+ ByteArrayOutputStream out = new ByteArrayOutputStream();
+ PrintStream stream = new PrintStream(out);
+ CarbonCli.run(args, stream);
+ String output = new String(out.toByteArray());
+ System.out.println(output);
Review comment:
avoid directly printing to console, use logger instead
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[hidden email]
With regards,
Apache Git Services