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

bugfix app-tomcat memory_pool unit mb #1268

Merged
merged 3 commits into from
Oct 8, 2023
Merged

Conversation

rbsrcy
Copy link
Contributor

@rbsrcy rbsrcy commented Oct 8, 2023

What's changed?

change app-tomcat memory_pool use MB as unit.

change app-tomcat memory_pool use MB as unit.

Checklist

  • I hereby agree to the terms of the HertzBeat CLA
  • I have read the Contributing Guide
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

@hertzbeat
Copy link
Contributor

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


This is my first submission, so I don’t quite understand it.

@tomsun28 tomsun28 changed the title 优化app-tomcat memory_pool 该用MB作为单位 bugfix app-tomcat memory_pool unit mb Oct 8, 2023
@tomsun28
Copy link
Contributor

tomsun28 commented Oct 8, 2023

@all-contributors please add @rbsrcy for code

@allcontributors
Copy link
Contributor

@tomsun28

I've put up a pull request to add @rbsrcy! 🎉

Comment on lines 187 to 194
# (optional)metrics field alias name, it is used as an alias field to map and convert the collected data and metrics field
# (可选)监控指标别名, 做为中间字段与采集数据字段和指标字段映射转换
aliasFields:
- Name
- Usage->committed
- Usage->init
- Usage->max
- Usage->used
Copy link
Contributor

Choose a reason for hiding this comment

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

hi 这里不能把 aliasFields参数去掉 这是从 Usage 拿子参数

Copy link
Contributor

@Ceilzcx Ceilzcx left a comment

Choose a reason for hiding this comment

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

LGTM

@tomsun28 tomsun28 merged commit ef4f725 into apache:master Oct 8, 2023
2 checks passed
tomsun28 pushed a commit that referenced this pull request Jan 16, 2024
tomsun28 pushed a commit that referenced this pull request Mar 9, 2024
tomsun28 pushed a commit that referenced this pull request Mar 9, 2024
tomsun28 pushed a commit that referenced this pull request Mar 10, 2024
tomsun28 pushed a commit that referenced this pull request Mar 10, 2024
tomsun28 pushed a commit that referenced this pull request Mar 11, 2024
tomsun28 pushed a commit that referenced this pull request Mar 11, 2024
tomsun28 pushed a commit that referenced this pull request Mar 11, 2024
tomsun28 pushed a commit that referenced this pull request Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants