You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2021/06/03 09:47:08 UTC

[GitHub] [incubator-doris] huzekang opened a new pull request #5968: Update install-deploy.md

huzekang opened a new pull request #5968:
URL: https://github.com/apache/incubator-doris/pull/5968


   improve doc to avoid error of installing be in hadoop cluster
   
   ## Proposed changes
   
   improve doc to avoid error
   
   ## Types of changes
   
   - [x ] Documentation Update (if none of the other choices apply)
   
   
   ## Checklist
   
   - [x ] If these changes need document changes, I have updated the document
   
   


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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org

[GitHub] [incubator-doris] morningman commented on a change in pull request #5968: Update install-deploy.md

Posted by GitBox <gi...@apache.org>.
morningman commented on a change in pull request #5968:
URL: https://github.com/apache/incubator-doris/pull/5968#discussion_r645983713



##########
File path: docs/zh-CN/installing/install-deploy.md
##########
@@ -159,7 +159,7 @@ BROKER 当前没有，也不需要 priority\_networks 这个选项。Broker 的
 
 * 修改所有 BE 的配置
 
-    修改 be/conf/be.conf。主要是配置 `storage_root_path`：数据存放目录。默认在be/storage下，需要**手动创建**该目录。多个路径之间使用 `;` 分隔（最后一个目录后不要加 `;`）。
+    修改 be/conf/be.conf。主要是配置 `storage_root_path`：数据存放目录。默认在be/storage下，需要**手动创建**该目录。多个路径之间使用 `;` 分隔（最后一个目录后不要加 `;`）。如果be部署在hadoop集群中，注意调整be.conf中的`webserver_port = 8040`,以免造成断开冲突

Review comment:
       ```suggestion
       修改 be/conf/be.conf。主要是配置 `storage_root_path`：数据存放目录。默认在be/storage下，需要**手动创建**该目录。多个路径之间使用 `;` 分隔（最后一个目录后不要加 `;`）。如果 BE 节点和 hadoop 组件混部，注意调整be.conf中的`webserver_port = 8040`,以免造成端口冲突
   ```
   
   Thanks for your PR. And please update the english document




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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org

[GitHub] [incubator-doris] morningman merged pull request #5968: Update install-deploy.md

Posted by GitBox <gi...@apache.org>.
morningman merged pull request #5968:
URL: https://github.com/apache/incubator-doris/pull/5968


   


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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org

[GitHub] [incubator-doris] huzekang commented on pull request #5968: Update install-deploy.md

Posted by GitBox <gi...@apache.org>.
huzekang commented on pull request #5968:
URL: https://github.com/apache/incubator-doris/pull/5968#issuecomment-855239642


   OK, I will update English doc later
   
   
   
   发自我的iPhone
   
   
   ------------------ Original ------------------
   From: Mingyu Chen ***@***.***&gt;
   Date: Sat,Jun 5,2021 8:14 PM
   To: apache/incubator-doris ***@***.***&gt;
   Cc: huzk ***@***.***&gt;, Author ***@***.***&gt;
   Subject: Re: [apache/incubator-doris] Update install-deploy.md (#5968)
   
   
   
   
   
    
   @morningman commented on this pull request.
    
    
   In docs/zh-CN/installing/install-deploy.md:
    &gt; @@ -159,7 +159,7 @@ BROKER 当前没有，也不需要 priority\_networks 这个选项。Broker 的    * 修改所有 BE 的配置   -    修改 be/conf/be.conf。主要是配置 `storage_root_path`：数据存放目录。默认在be/storage下，需要**手动创建**该目录。多个路径之间使用 `;` 分隔（最后一个目录后不要加 `;`）。 +    修改 be/conf/be.conf。主要是配置 `storage_root_path`：数据存放目录。默认在be/storage下，需要**手动创建**该目录。多个路径之间使用 `;` 分隔（最后一个目录后不要加 `;`）。如果be部署在hadoop集群中，注意调整be.conf中的`webserver_port = 8040`,以免造成断开冲突  ⬇️ Suggested change -    修改 be/conf/be.conf。主要是配置 `storage_root_path`：数据存放目录。默认在be/storage下，需要**手动创建**该目录。多个路径之间使用 `;` 分隔（最后一个目录后不要加 `;`）。如果be部署在hadoop集群中，注意�
 �整be.conf中的`webserver_port = 8040`,以免造成断开冲突 +    修改 be/conf/be.conf。主要是配置 `storage_root_path`：数据存放目录。默认在be/storage下，需要**手动创建**该目录。多个路径之间使用 `;` 分隔（最后一个目录后不要加 `;`）。如果 BE 节点和 hadoop 组件混部，注意调整be.conf中的`webserver_port = 8040`,以免造成端口冲突  
   Thanks for your PR. And please update the english document
    
   —
   You are receiving this because you authored the thread.
   Reply to this email directly, view it on GitHub, or unsubscribe.


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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org