You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by jackylk <gi...@git.apache.org> on 2017/09/14 09:00:56 UTC

[GitHub] carbondata pull request #1357: [CARBONDATA-1478]update compaction documentat...

Github user jackylk commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1357#discussion_r138835309
  
    --- Diff: docs/data-management.md ---
    @@ -135,7 +135,7 @@ CLEAN FILES FOR TABLE table1
        
     | Parameter | Default | Application | Description | Valid Values |
     |-----------------------------------------|---------|-------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|--------------|
    -| carbon.compaction.level.threshold | 4, 3 | Minor | This property is for minor compaction which decides how many segments to be merged. Example: If it is set as 2, 3 then minor compaction will be triggered for every 2 segments. 3 is the number of level 1 compacted segment which is further compacted to new segment. | NA |
    +| carbon.compaction.level.threshold | 4, 3 | Minor | This property is for minor compaction which decides how many segments to be merged. Example: If it is set as 2, 3, then minor compaction will be triggered for every 2 segments in level 1. 3 is the number of level 1 compacted segment which is further compacted to new segment in level 2. | NA |
    --- End diff --
    
    suggest to change `2, 3` to `"2, 3"`


---