Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2462#discussion_r202229386 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/datacompaction/CarbonIndexFileMergeTestCase.scala --- @@ -193,6 +196,30 @@ class CarbonIndexFileMergeTestCase sql("select * from mitable").show() } + // CARBONDATA-2704, test the index file size after merge + test("Verify the size of the index file after merge") { + sql("DROP TABLE IF EXISTS fileSize") + sql( + """ + | CREATE TABLE fileSize(id INT, name STRING, city STRING, age INT) + | STORED BY 'org.apache.carbondata.format' + | TBLPROPERTIES('SORT_COLUMNS'='city,name') + """.stripMargin) + sql(s"LOAD DATA LOCAL INPATH '$file2' INTO TABLE fileSize OPTIONS('header'='false')") + val table = CarbonMetadata.getInstance().getCarbonTable("default", "fileSize") + var loadMetadataDetails = SegmentStatusManager + .readTableStatusFile(CarbonTablePath.getTableStatusFilePath(table.getTablePath)) + var segment0 = loadMetadataDetails.filter(x=> x.getLoadName.equalsIgnoreCase("0")) + Assert.assertEquals(692, segment0.head.getIndexSize.toLong) --- End diff -- Do not assert for hard coded values. Instead create the file path and take its size using CarbonFile getSize interface method
---