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

    https://github.com/apache/carbondata/pull/3054#discussion_r246047276
  
    --- Diff: docs/documentation.md ---
    @@ -29,15 +29,15 @@ Apache CarbonData is a new big data file format for 
faster interactive query usi
     
     **Quick Start:** [Run an example 
program](./quick-start-guide.md#installing-and-configuring-carbondata-to-run-locally-with-spark-shell)
 on your local machine or [study some 
examples](https://github.com/apache/carbondata/tree/master/examples/spark2/src/main/scala/org/apache/carbondata/examples).
     
    -**CarbonData SQL Language Reference:** CarbonData extends the Spark SQL 
language and adds several [DDL](./ddl-of-carbondata.md) and 
[DML](./dml-of-carbondata.md) statements to support operations on it.Refer to 
the [Reference Manual](./language-manual.md) to understand the supported 
features and functions.
    +**CarbonData SQL Language Reference:** CarbonData extends the Spark SQL 
language and adds several [DDL](./ddl-of-carbondata.md) and 
[DML](./dml-of-carbondata.md) statements to support operations on it. Refer to 
the [Reference Manual](./language-manual.md) to understand the supported 
features and functions.
     
     **Programming Guides:** You can read our guides about [Java APIs 
supported](./sdk-guide.md) or [C++ APIs supported](./csdk-guide.md) to learn 
how to integrate CarbonData with your applications.
     
     
     
     ## Integration
     
    -CarbonData can be integrated with popular Execution engines like 
[Spark](./quick-start-guide.md#spark) , [Presto](./quick-start-guide.md#presto) 
and [Hive](./quick-start-guide.md#hive).Refer to the [Installation and 
Configuration](./quick-start-guide.md#integration) section to understand all 
modes of Integrating CarbonData.
    +CarbonData can be integrated with popular Execution engines like 
[Spark](./quick-start-guide.md#spark) , [Presto](./quick-start-guide.md#presto) 
and [Hive](./quick-start-guide.md#hive). CarbonData also supports read and 
write with [Alluxio](./quick-start-guide.md#alluxio). Refer to the 
[Installation and Configuration](./quick-start-guide.md#integration) section to 
understand all modes of Integrating CarbonData.
    --- End diff --
    
    I think it's not proper to mention Alluxio after e(*Not E*)xecution engines 
like SparkSQL/Presto/Hive.
    
    Meanwhile we can add another paragraph and mention CarbonData can integrate 
with other storage engines such as HDFS, S3, OBS, Alluxio.
    
    @chenliang613 How do you think about it?


---

Reply via email to