qxo
> 感觉 slf4j-simple 的情况很少？有无必要加上？ 肯定很少看到在用，但手头有实际的项目也确实在用，所以就加了：） 对于大部人不需要，但有些环境又需要的功能，我们现在是否已可用扩展配置或SPI方式来支持？
> 下面两个场景会产生较多内存碎片： > 1）`sc -d` 查询类详细信息，每个类需要创建一个vo > 2）`sm` 每个方法会创建一个vo > > 而对于 `sc` 查询结果数量很多也不会产生大量内存碎片，只是一个size很大的List列表，也不会新增String对象，不需要限制。 有时需要使用组合命令查询过滤，限制数量会导致这种方式不能用了。 就算不会占用太多CPU和内存，但让CPU做无用功总是事实：假设有10万个符合要求的类，实际只要显示100就够了：那为什么我们先找出这所有10万个，然后我们只显示其中的100个？ 原来SearchUtils内部的filter实现也是这样：先找匹配的再过滤，一次性匹配过滤所要的不是更直接：） we can do better，why not ?
why？ 这里应该有个解释... 还是出于你们某位领导的一个执念... ------------------ 原始邮件 ------------------ 发件人: "gongdewei"
> 不能影响这种组合命令的使用： `sc *List | wc -l` 已支持，请看提交日志：）
这是不是sm不-n就无效了！！！ 我只想让工具功能更强... 你这么执着于你原来逻辑...那随你吧... 你想缩小原PR的能力，我保留意见，自己改吧：） ------------------ 原始邮件 ------------------ 发件人: "gongdewei"
？ 怎么个无效 ？ 我认为git提交日志已解释清楚了... ``` feat: 尊重PR审核方意见改进了numberOfLimit默认值逻辑 1. 当isDetail=true时， 如numberOfLimit>100 或 numberOfLimit
ok:) 我先把此PR和主分区同步检一下先:) ------------------ 原始邮件 ------------------ 发件人: "hengyunabc"
@hengyunabc 我已重新处理过了：基于最新主分支然后重新git cherry-pick此PR, 请审查代码：） 欢迎指正：）
另外[原PR](https://github.com/alibaba/arthas/pull/1291/files） sm是-n参数是限制方法个名的，本PR重新合并时没加个功能：也就是说此PR当前逻辑sm还只是限制类的个数的， 这个你们看一下：要或不要改
> RepositoryChangeListener 应该没法实现这个需求吧，可行的方式可以参考 com.ctrip.framework.apollo.config.data.internals.PureApolloConfigFactory，通过 ApolloInjectorCustomizer 来自定义一个 Config 类实现出来 经过此PR， 是可能实现对apollo中获取配置项进行解码处理再交由程序来处理。 API应提供扩展点以便具体项目进行个性化改造（而不用修改源码） 以下是切入点调用链供参考 //启动时的SPI切入点： ``` at com.ctrip.framework.apollo.internals.AbstractConfigRepository.fireRepositoryChange(AbstractConfigRepository.java:53) ~[apollo-client-1.8.0.jar:1.8.0] at com.ctrip.framework.apollo.internals.RemoteConfigRepository.sync(RemoteConfigRepository.java:148) ~[classes/:1.8.0] at com.ctrip.framework.apollo.internals.AbstractConfigRepository.trySync(AbstractConfigRepository.java:25) ~[apollo-client-1.8.0.jar:1.8.0] at com.ctrip.framework.apollo.internals.RemoteConfigRepository.(RemoteConfigRepository.java:97) ~[classes/:1.8.0] at com.ctrip.framework.apollo.spi.DefaultConfigFactory.createRemoteConfigRepository(DefaultConfigFactory.java:77) ~[apollo-client-1.8.0.jar:1.8.0] at...