Github user gvramana commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2190#discussion_r183225467
--- Diff:
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CarbonReaderTest.java ---
@@ -48,11 +48,11 @@ public void testWriteAndReadFiles() throws IOException,
InterruptedException {
int i = 0;
while (reader.hasNext()) {
- Object[] row = (Object[])reader.readNextRow();
- Assert.assertEquals("robot" + (i % 10), row[0]);
- Assert.assertEquals(i, row[1]);
+ Object[] row = (Object[]) reader.readNextRow();
+ // TODO: Default sort column is applied for dimensions. So, need to
validate accordingly
--- End diff --
TODO: to be corrected
---
