dingshuangxi888 commented on PR #7725: URL: https://github.com/apache/rocketmq/pull/7725#issuecomment-2002982544
> 1. 缓存部分，打开 cache 自带的 stats 统计能力并输出，性能影响很小，默认缓存大小是否合理？ 先试运行一段时间，后期考虑可以加一下 > 2. 能否快速找到问题用户或者资源？ 目前通过审计日志进行支持 > 3. this.kvStore.flushWal(true); rocksdb 部分应该保证数据同步落盘。 底层方法使用的就是这个能力 > 4. 鉴权降级和部分降级能力是否验证过。 可通过修改配置进行降级 > 5. RequestHeaderRegistry.getInstance().initialize(); 这个地方能不能去单例，broker container 模式是否可以？ 这个功能是扫描所有的RequestHeader然后构建一个映射关系，这个系统级别的数据，用单例还是比较合适的 > 6. if (this.authorizationMetadataManager != null) 这里 bug 了，shutdown 了两次。。。 已修复，感谢指出 > 7. 请求失败时，错误日志过多，会打爆磁盘。 目前通过日志级别来进行日志区分，后面可以有更加高级的策略进行日志打印，支持问题有效排查的同时，避免日志过多 > 8. RemotingCode 需要加前缀和别的请求区分。 已修复，感谢指出 > 9. AbstractTransactionalMessageCheckListener.java 这里加个 topic 干啥用的？ 事务相关的API原来没有透传topic，导致无法在请求面解析用户资源。最新客户端SDK已经加上这个字段 > 10. GrpcServerBuilder.java 鉴权 interceptors 和 global interceptors 的顺序是否正确？ interceptor就是倒叙添加的，这个已经检查并确认 -- 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. To unsubscribe, e-mail: commits-unsubscr...@rocketmq.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
