[GitHub] [carbondata] PurujitChaugule commented on a change in pull request #3917: [CARBONDATA-3978] Clean Files Refactor and support for trash folder in carbondata

[GitBox <gi...@apache.org>]

PurujitChaugule commented on a change in pull request #3917:
URL: https://github.com/apache/carbondata/pull/3917#discussion_r509201958



##########
File path: docs/dml-of-carbondata.md
##########
@@ -552,3 +553,50 @@ CarbonData DML statements are documented here,which includes:
   ```
   CLEAN FILES FOR TABLE carbon_table
   ```
+
+## CLEAN FILES
+
+  Clean files command is used to remove the Compacted and Marked
+  For Delete Segments from the store. Carbondata also supports Trash
+  Folder where all the stale data is moved to after clean files
+  is called
+
+  There are several types of compaction
+
+  ```
+  CLEAN FILES ON TABLE TableName
+  ```

Review comment:
       Clean files syntax needs to be changed from "clean files on table tablename" to  "clean files for table tablename" as testcases mentioned use  the above




----------------------------------------------------------------
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:
users@infra.apache.org