Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MigrationCost] 搬移時間限制 實驗報告 #1772

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

qoo332001
Copy link
Collaborator

此PR新增 #1665 的實驗報告

@qoo332001 qoo332001 requested a review from chia7712 May 25, 2023 16:08
@qoo332001 qoo332001 changed the title [MigrationCost] 搬移時間限制限制 實驗報告 [MigrationCost] 搬移時間限制 實驗報告 May 26, 2023
Copy link
Contributor

@chia7712 chia7712 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@qoo332001 感謝實驗

中文的描述可以再多讀幾次,盡量精練和精準,這練習一下對自己寫論文也會有好處


## 成本估計

* 成本估計實驗報告
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

同樣跟標題重複了


* 成本估計實驗報告
* [磁碟空間限制實驗](experiment_brokerDiskSpace.md) : kafka partition的搬移過程中會產生一些成本,在搬移前先估計出搬移partition過程中可能佔用的broker/硬碟空間並對其做限制,確保搬移不會佔用過多的儲存空間
* [搬移時間限制實驗](experiment_partitionMigrateTime.md) : kafka partition的搬移過程中會產生一些成本,在搬移前先估計出搬移partition過程中可能花費多少搬移時間,並對其做限制確保搬移不會花費過多的時間
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確保搬移不會花費過多的時間 這句的描述也需要調整,我們無法保證不會花費“過多”的時間,只能保證不會超過使用者定義的時間上限

進行底層硬體效能資料的攝取。

您可以使用專案內的
[./docker/start_node_exporter.sh](https://github.com/skiptests/astraea/blob/7596f590ae0f0ec370a6e257c10cc2aeb5fb5bf4/docs/run_node_exporter.md),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

這裡的縮寫可以直接說 node exporter 就好,剩下的就去參考連結中查看

}
],
"moveCosts": [
"org.astraea.common.cost.PartitionMigrateTimeCost"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

這個名稱我在 #1665 有提到,可以更簡化一點

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

Successfully merging this pull request may close these issues.

2 participants