You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2020/01/02 06:46:03 UTC

[GitHub] [incubator-iotdb] linliu03 opened a new pull request #703: Edited the English for Readme.md

linliu03 opened a new pull request #703: Edited the English for Readme.md
URL: https://github.com/apache/incubator-iotdb/pull/703
 
 
   I polish the content of the Readme.md to fix some typos..

----------------------------------------------------------------
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


With regards,
Apache Git Services

[GitHub] [incubator-iotdb] qiaojialin merged pull request #703: Edited the English for Readme.md

Posted by GitBox <gi...@apache.org>.

qiaojialin merged pull request #703: Edited the English for Readme.md
URL: https://github.com/apache/incubator-iotdb/pull/703
 
 
   

----------------------------------------------------------------
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


With regards,
Apache Git Services

[GitHub] [incubator-iotdb] jixuan1989 commented on a change in pull request #703: Edited the English for Readme.md

Posted by GitBox <gi...@apache.org>.

jixuan1989 commented on a change in pull request #703: Edited the English for Readme.md
URL: https://github.com/apache/incubator-iotdb/pull/703#discussion_r362383002

##########
File path: README.md
##########
@@ -32,21 +32,21 @@

# Overview

-IoTDB (Internet of Things Database) is an integrated data management engine designed for time series data, which can provide users specific services for data collection, storage and analysis. Due to its light weight structure, high performance and usable features together with its intense integration with Hadoop and Spark ecology, IoTDB meets the requirements of massive dataset storage, high-speed data input and complex data analysis in the IoT industrial field.
+IoTDB (Internet of Things Database) is an integrated data management engine designed for time series data, which can provide users specific services for data collection, storage and analysis. Due to its light weight structure, high performance and usable features together with its intense integration with Hadoop and Spark ecology, IoTDB meets the requirements of massive dataset storage, high throughput data input and complex data analysis in the industrial IoTDB field.

# Main Features

-IoTDB's features are as following:
+Main features of IoTDB are as follows:

-1. Flexible deployment. IoTDB provides users one-click installation tool on the cloud, once-decompressed-used terminal tool and the bridge tool between cloud platform and terminal tool (Data Synchronization Tool).
-2. Low cost on hardware. IoTDB can reach a high compression ratio of disk storage
+1. Flexible deployment strategy. IoTDB provides users one-click installation tool on the cloud, one-shot compression terminal tool and a data synchronization tool bridging cloud platform and terminal.

Review comment:
Hm... I can not understand the original and the modified sentences....
I think if we want to highlight the "flexible deployment strategy" feature, we need to show two points:
1. IoTDB provides users one-click installation tool either on the cloud or the terminal device (maybe "embedded device" is better.) (users just need to using docker command or unzip IoTDB's zip file to enjoy IoTDB).
2. We provide interactive CLI tool to operate IoTDB, and a data synchronization tool bridging cloud platform and terminal device (or embedded device).

----------------------------------------------------------------
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

With regards,
Apache Git Services

[GitHub] [incubator-iotdb] linliu03 commented on a change in pull request #703: Edited the English for Readme.md

Posted by GitBox <gi...@apache.org>.

linliu03 commented on a change in pull request #703: Edited the English for Readme.md
URL: https://github.com/apache/incubator-iotdb/pull/703#discussion_r362383417
 
 

 ##########
 File path: README.md
 ##########
 @@ -255,7 +255,7 @@ IoTDB> SELECT status FROM root.ln.wf01.wt01
 Total line number = 2
 ```
 
-We can also query several timeseries data at once like this:
+We can also query several timeseries data at the same time:
 
 Review comment:
   Yeah, I think this clearly states the purpose of the command. 

----------------------------------------------------------------
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


With regards,
Apache Git Services

[GitHub] [incubator-iotdb] jixuan1989 commented on a change in pull request #703: Edited the English for Readme.md

Posted by GitBox <gi...@apache.org>.

jixuan1989 commented on a change in pull request #703: Edited the English for Readme.md
URL: https://github.com/apache/incubator-iotdb/pull/703#discussion_r362382016
 
 

 ##########
 File path: README.md
 ##########
 @@ -255,7 +255,7 @@ IoTDB> SELECT status FROM root.ln.wf01.wt01
 Total line number = 2
 ```
 
-We can also query several timeseries data at once like this:
+We can also query several timeseries data at the same time:
 
 Review comment:
   how about "We can also query several timeseries data using one SQL statement"

----------------------------------------------------------------
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


With regards,
Apache Git Services