Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2807#discussion_r228572221
--- Diff:
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CarbonSchemaReaderTest.java
---
@@ -0,0 +1,233 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
--- End diff --
This test case is test for new class, like Schema. java and we should move
read schema test case from CarbonReaderTest to CarbonSchemaReaderTest---
