Skip to content
This repository has been archived by the owner on Jan 7, 2023. It is now read-only.

do analyzer at nodes #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Shaoshian
Copy link
Contributor

Signed-off-by: shaoshian [email protected]

更新后需要增加环境的配置:

1.node需要安装与head相同的包依赖。
2.node的/home/下面需要放置一个最新版本的Cetune工具。

@Shaoshian Shaoshian force-pushed the do_analyzer_at_node branch from 60e4274 to ad895a0 Compare December 1, 2016 06:31
Signed-off-by: shaoshian <[email protected]>
@Shaoshian Shaoshian force-pushed the do_analyzer_at_node branch from ad895a0 to ae47714 Compare December 1, 2016 08:16
@Shaoshian
Copy link
Contributor Author

@xuechendi hi 你确认一下吧。。

@xuechendi
Copy link
Contributor

@Shaoshian, please verify the codes in your UP cluster, to finish 10.2.3 test, I will merge the codes once UP test done. And also please ask rentao to try the codes on his cluster firstly.

@Shaoshian
Copy link
Contributor Author

@xuechendi 我跑了5-6个case 正常,如果rentao那边遇到问题的话,我会让他直接反馈给我,然后修改的。。

@Shaoshian
Copy link
Contributor Author

@xuechendi 你也可以给原来做一个备份,如果,我的PR有什么问题的话,可以回到以前的版本,这样就可以确保有一个OK的版本的。。

@xuechendi
Copy link
Contributor

@Shaoshian , since this is a big PR, I won't merge to upstream/master now. Can you send a PR to the new branch wip_distributed_analyze? I will leave codes there, and merge it after we done the UP test

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants