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

[ROCKETMQ-289] getBrokerRuntimeInfo method output fileReservedTime value #163

Closed
wants to merge 1 commit into from
Closed

Conversation

fuyou001
Copy link
Contributor

Motivation:
add fileReservedTime value for getBrokerRuntimeInfo return runtimeInfo

Modification:
getBrokerRuntimeInfo method add a key fileReservedTimeByHour mean fileReservedTime in runtimeInfo

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 38.851% when pulling c8079c3 on fuyou001:ROCKETMQ-289 into 368e7c8 on apache:develop.

@zhouxinyu
Copy link
Member

Hi, thanks for your contribution. We could get FileReservedTime through GetBrokerConfigCommand, there is no need to add it to runtimeInfo .

@fuyou001 fuyou001 closed this Sep 15, 2017
lizhanhui pushed a commit to lizhanhui/rocketmq that referenced this pull request Jun 25, 2019
1. support for modifying the config of name server dynamically
2. add two request codes 3. add two commands
3. set config store path when start up using special property file
lizhanhui pushed a commit to lizhanhui/rocketmq that referenced this pull request Jun 25, 2019
1. format file header
lizhanhui pushed a commit to lizhanhui/rocketmq that referenced this pull request Jun 25, 2019
1. Directly print msg after query by queue offset.
lizhanhui pushed a commit to lizhanhui/rocketmq that referenced this pull request Jun 25, 2019
1. merge master
lizhanhui pushed a commit to lizhanhui/rocketmq that referenced this pull request Jun 25, 2019
1. command of get broker config
lizhanhui pushed a commit to lizhanhui/rocketmq that referenced this pull request Jun 25, 2019
Feature/namesrv config 20161115

see apache#163 

see apache#158 


See merge request !23
lollipopjin added a commit to lollipopjin/rocketmq that referenced this pull request Jun 16, 2022
…ature_1

* origin/develop_with_prehistory: (3800 commits)
  Issue apache#179 [BUG]check properties length
  Issue apache#161 printmsg bug 修复
  Issue apache#175 change version to 3.6.3-SNAPSHOT
  Issue apache#175 pub 3.6.2 server release
  Issue apache#174 顺序消息重试次数超过默认值,导致消息直接进入死信队列
  Issue apache#173 日志滚动异常,更新日志配置
  Issue apache#172 rebalance 抛 Error 异常,导致 rebalance 被中断
  Issue apache#171 指定消息 id 进行重新发送
  Issue apache#146 add trace hook switch and consume return type
  Issue apache#146 add trace hook switch and consume return type
  Issue apache#169
  Issue apache#158 1. Unify config manager to configuration in common. 2. Configuration is hold in controller. 3. Extend config object or property can register to configuration.
  Issue apache#163 1. command of get broker config
  Issue apache#163 1. Directly print msg after query by queue offset.
  Issue apache#163 1. format file header
  Issue apache#163 1. support for modifying the config of name server dynamically 2. add two request codes 3. add two commands 3. set config store path when start up using special property file
  Issue apache#137 【买卖家】队列热点数据分析
  Issue apache#162 ConsumerConnection 获取连接失败。
  Issue apache#161 printmsg 使用 tag 过滤时,拉消息不全
  Issue apache#160 重置位点优化
  ...
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.

None yet

3 participants