chiangzeon

Results 21 comments of chiangzeon

> [@Saljack](https://github.com/Saljack), you can use following `jni-config.json` > > [ > { > "name":"sun.net.dns.ResolverConfigurationImpl", > "fields":[ > {"name":"os_searchlist"}, > {"name":"os_nameservers"} > ] > } > ] > in conjunction with...

> 最近已将修复程序合并到 ResolverConfigurationImpl 中,请参阅[#10396](https://github.com/oracle/graal/pull/10396) > > 我假设这个问题已经解决,类似于[#9597。](https://github.com/oracle/graal/issues/9597) > > (但您可能需要我们主分支的 EA 版本或从主分支构建的版本)。 I got it,thank you

> 通过在 NacosConfigManager 的 bean 初始化完成之前,从整个 env 中重新刷新配置 你好,“通过在 NacosConfigManager 的 bean 初始化完成之前,从整个 env 中重新刷新配置”这段我不是很理解,能解释下什么意思么

> > 通过在 NacosConfigManager 的 bean 初始化完成之前,从整个 env 中重新刷新配置 > > 你好,“通过在 NacosConfigManager 的 bean 初始化完成之前,从整个 env 中重新刷新配置”这段我不是很理解,能解释下什么意思么 > > > 通过在 NacosConfigManager 的 bean 初始化完成之前,从整个 env 中重新刷新配置 > >...

> > > 通过在 NacosConfigManager 的 bean 初始化完成之前,从整个 env 中重新刷新配置 > > > > > > 你好,“通过在 NacosConfigManager 的 bean 初始化完成之前,从整个 env 中重新刷新配置”这段我不是很理解,能解释下什么意思么 > > > > > 通过在 NacosConfigManager...

> [@chiangzeon](https://github.com/chiangzeon) 这位参与讨论的开发者,你怎么看呢? 我理解下来的意思,nacosconfigmanager确实存在于spring context中,但它处于一个中间态,有这个实例,但变量是未加载的,本次pr的目的,其实是希望处于一个强一致性。 情况1、如未配置import,则context中不存在NacosConfigManager 情况2、如未配置import,则按照已知内容,如server-addr等其他属性去加载,保证NacosConfigManager对象的可用性 本次pr的目的是情况2。不知是否看法一致

> > > [@chiangzeon](https://github.com/chiangzeon) 这位参与讨论的开发者,你怎么看呢? > > > > > > 我理解下来的意思,nacosconfigmanager确实存在于spring context中,但它处于一个中间态,有这个实例,但变量是未加载的,本次pr的目的,其实是希望处于一个强一致性。 情况1、如未配置import,则context中不存在NacosConfigManager 情况2、如未配置import,则按照已知内容,如server-addr等其他属性去加载,保证NacosConfigManager对象的可用性 本次pr的目的是情况2。不知是否看法一致 > > 正是,完全是你理解的一样。 > > 所以库要么符合1 即不该存在这个实例。 要么符合2 即存在了就应该是正确的。PR解决了2的错误问题并没有改变设计初衷。 但是有个点尚未提及,就是spring-cloud-starter-bootstrap依赖或spring.cloud.bootstrap.enabled、spring.config.use-legacy-processing属性,因为按现有流程,如果缺失该依赖或属性设置为true,在缺少import配置时,应用将无法启动,代码块见org.springframework.cloud.commons.ConfigDataMissingEnvironmentPostProcessor#postProcessEnvironment @uuuyuqi ```xml org.springframework.cloud spring-cloud-starter-bootstrap...

> > > > > [@chiangzeon](https://github.com/chiangzeon) 这位参与讨论的开发者,你怎么看呢? > > > > > > > > > > > > 我理解下来的意思,nacosconfigmanager确实存在于spring context中,但它处于一个中间态,有这个实例,但变量是未加载的,本次pr的目的,其实是希望处于一个强一致性。 情况1、如未配置import,则context中不存在NacosConfigManager 情况2、如未配置import,则按照已知内容,如server-addr等其他属性去加载,保证NacosConfigManager对象的可用性 本次pr的目的是情况2。不知是否看法一致 > > > > > >...

> 我这边最近没有时间处理这个了,我只按照上文的第2点的一致性问题解决了第2点的一致性问题。 > > 现在结论选择第1点来解决一致性问题,如果 [@chiangzeon](https://github.com/chiangzeon) 方便的话可以处理,或者由其他同学处理。 好的,我来看下

> > 你好,我想你们可能没有真正的debug验证。现在并不是如你所说没有import就不会有nacosconfigmanager对象。就算没有配置任何import目前也是能注入bean对象的。你们要么如你所说不import就不初始化它那也没有问题。现在实际情况是初始化了一个错误的。 我的修改并没有增加额外的初始化,pr的代码仅仅是确保属性的准确性,所以它应该属于代码完善或优化而不属于改变规则。 > > 我本身也是在nacos的社区群里,只不过不在这个cloud群里。希望你们能理解我的意思,或者有微信等其他方式 进一步讨论。 > > 抱歉,可能我前面没有说明清楚,我想表达的是:在没有配置 spring.config.import 的时候,NacosConfigManager 这个 bean 拿不到 config 数据是符合预期的,即便此时 NacosConfigManager 这个 bean 存在于 Spring Context 中且可以被编程方式使用。至少设计初衷上,spring.config.import 一定是必需的。(值得一提的是,SCA 对 SpringCloud Bootstrap 相关的适配内容,会在后续版本中删除,目前不会再去主动维护,也不推荐继续使用该功能)...