Xz
Xz
> 对应的spring-boot版本没有更新呢? 依赖包nacos-config-spring-boot-autoconfigure 另外,nacos-config-spring-boot-starter目前最新0.3.0-RC版本,何时发布最新版本? 最后,使用nacos客户端 spring-boot项目java -jar启动时使用时需要避免使用ForkJoinPool线程池,因为其内部使用的系统(应用)类加载器,会导致无法加载相关类 可以考虑下替换为`spring-cloud-alibaba`, 看`spring-boot`版本社区很久没更新了 活跃度都不怎么高
非常感谢大佬指出的问题,第一次参与开源项目PR让我认识到自身还有很多思考量不足,后续会继续学习,争取能为社区贡献。
> > 已知问题,请升级版本。 > > #10674 > > 请问为什么抛出NPE异常的原因跟#10674有关? `StringPool` 可能导致NPE
> **Describe the bug** step1:在某次版本升级过程中,下午4点50多的时候,在Nacos集群中,通过web端,登录Nacos,更改了Redis的相关配置,由Redis的单节点配置修改为Redis哨兵模式,并且确认,redis配置已经修改成功 step2:等到晚上7点多的时候,在升级应用的过程中,发现应用节点无法启动,日志报错为连接Redis失败;此时去到Nacos中,确认Redis配置正确; step3:应用经过自动重启后,启动成功;(有的应用重启一次就能启动成功,有的应用需要重启2~3次才能成功) step4:经过仔细排查应用日志,发现应用在首次启动时,无法正常启动的日志,拉取的配置,依旧是Redis单节点的配置(原来的配置);启动成功的日志,拉取到的配置是Redis哨兵配置(正常配置) step5:对启动成功的应用,再次重启,还是需要多次重启,才能启动成功 step6:此时,再次去到Nacos端,登录Nacos后发现,Redis的配置正常的;此时又重新点击了两次发布的按钮 step7:再次重启应用,可以一次启动成功 > > 此时,大概定位到Nacos节点配置同步问题,于是,去到Nacos端,查看Nacos日志,在不同的Nacos节点,发现了不同的日志: > > ``` > 2024-07-30 16:57:30,669 INFO [dump-task] add task. groupKey=redis.properties+DEFAULT_GROUP+391d86d8-dbdf-4580-aa63-af06225007c3, taskKey=redis.properties+DEFAULT_GROUP+391d86d8-dbdf-4580-aa63-af06225007c3+false > > 2024-07-30 16:57:30,685...
第一次尝试提交PR, 希望大佬能够review指点一下, 非常感谢🙏
更新了一下, 也烦请大佬有空review一下
> 感觉上问题已经快收敛完了, 再看下评论进行一次修改。 > > 另外有一个问题可以思考下: > > moduleName时通过单独的参数传递进去还是方properties里比较好? 我感觉放properties有一点晦涩,直接传进去是不是明显一点。 确实, 看一下是有点晦涩,这个属性我觉得应该是必要属性要依赖方感知和设置这个属性, 放properties的话貌似还需要去理解上下文这个属性的作用, 假设另一个模块引用的话可能是不知道这个属性的.
更正了一版, 对于`ServerHttpAgent`的改动之前有些问题 这里修复了, 配置中心保持一个访问同一只可访问的server的逻辑是保留了的,在`genNextServer()`内部就已经修改当前服务下标并返回的, 之前是采用的迭代器方式才额外调用了方法修改当前服务. @KomachiSion 也麻烦大佬有空了看一下有什么问题
> @XiaZhouxx 您好, 我收到主办方的消息, 说您的PR没有在活动官网上登记, 这样的话这个PR不算是有效的参赛PR, > > 方便抽空把报名信息在官网上填写一下。 > > https://tianchi.aliyun.com/competition/entrance/532215/information?spm=a2c22.12281976.0.0.41ab6c71nLISOm > > 对填写有疑问也可以发邮件到 [[email protected]](mailto:[email protected]) 询问。 收到, 已经填报PR, 代码也重新格式化排版 @KomachiSion 也烦请大佬抽空看一下
> 目前初步review似乎没有太大的问题了, 之后会在活动截止前对PR进行整体评判后决定是否合入。 > > 我先跑一下CI,请关注一下结果,避免后续需要合入时由于CI问题被阻塞。 好的, 辛苦了