为什么 PropertiesCompatibleFileConfigRepository 不适合 Properties 类型的配置？

[yfwz100]

我观察到有一些 DefaultConfigFactory#create() 方法在创建 Config 的时候有一个注释：

apollo-java/apollo-client/src/main/java/com/ctrip/framework/apollo/spi/DefaultConfigFactory.java

Lines 77 to 80 in a9a6edc

	// although ConfigFileFormat.Properties are compatible with themselves we
	// should not create a PropertiesCompatibleFileConfigRepository for them
	// calling the method `createLocalConfigRepository(...)` is more suitable
	// for ConfigFileFormat.Properties

我查看了 PropertiesCompatibleFileConfigRepository 的实现，这是直接监听 ConfigFile 的变更来实现更新，对于 Properties 类型的配置，有什么特别的吗？

[nobodyiam]

结论：PropertiesCompatibleFileConfigRepository 主要是给 YAML/YML 这类“文件内容需要先转换成 Properties”的格式用的，Properties 类型本身已经是 key-value Properties，所以 Config#getProperty 读取时应走普通 ConfigRepository 链路，而不是再绕一层 ConfigFile 适配。

当前链路大致是：

• DefaultConfigFactory#create(...) 会先通过 determineFileFormat(namespace) 判断格式。只有 ConfigFileFormat.isPropertiesCompatible(format) 且 format != ConfigFileFormat.Properties 时，才会创建 PropertiesCompatibleFileConfigRepository。
• 对 Properties 类型，RemoteConfigRepository#getConfig() 已经直接把 ApolloConfig#getConfigurations() 转成 Properties，DefaultConfig 可以直接消费这份 key-value 数据。
• 对 YAML/YML 类型，服务端返回的是文件内容，客户端会通过 PlainTextConfigFile#getContent() 读取 content 字段，再由 YamlConfigFile#asProperties() 调用 YamlParser#yamlToProperties(...) 转成扁平化的 Properties，所以这里需要 PropertiesCompatibleFileConfigRepository 把 ConfigFile 适配回 ConfigRepository。

所以这里的“特别”主要不是 Properties 不兼容，而是 Properties 不需要转换层。PropertiesConfigFile 仍然有意义，它用于 ConfigService.getConfigFile(..., ConfigFileFormat.Properties) 这种文件视角 API，可以把 Properties 渲染成文本内容；但对 ConfigService.getConfig(...) 返回的 Config 来说，直接使用普通 repository 更简单，也避免创建额外的 ConfigFile -> ConfigRepository 包装链路。

代码上可以参考：

• apollo-client/src/main/java/com/ctrip/framework/apollo/spi/DefaultConfigFactory.java
• apollo-client/src/main/java/com/ctrip/framework/apollo/internals/RemoteConfigRepository.java
• apollo-client/src/main/java/com/ctrip/framework/apollo/internals/PlainTextConfigFile.java
• apollo-client/src/main/java/com/ctrip/framework/apollo/internals/YamlConfigFile.java
• apollo-client/src/main/java/com/ctrip/framework/apollo/internals/PropertiesConfigFile.java

另外，当前注释里提到 createLocalConfigRepository(...)，从现在的代码看更准确的说法应该是走普通的 createConfigRepository(...) 链路，因为它会按配置选择 Kubernetes cache、本地文件 cache 或远端 repository。

[yfwz100]

感谢回答！不过从这个回答上看，感觉抽象一个 FileContentLike 的接口更合适，PropertiesCompatibleFileConfigRepository 对应的是纯文本更新的配置类型。