Apache Kyuubi项目中Trino引擎配置层级问题分析

Apache Kyuubi是一个开源的分布式SQL查询引擎，它提供了统一的JDBC接口来访问不同的计算引擎。在Kyuubi项目中，Trino引擎作为其中一个重要的支持引擎，近期发现了一个关于配置层级的重要问题。

问题背景

在Kyuubi的Trino引擎实现中，存在一个配置层级处理不当的问题。具体表现为在TrinoSessionImpl类的实现中，错误地将引擎级别的配置当作会话级别的配置来使用。这个问题会影响Trino引擎在多会话环境下的配置隔离性。

技术细节分析

在当前的实现中，TrinoSessionImpl类在初始化时会从sessionManager获取配置，并将其视为会话级别的配置。然而，实际上sessionManager.getConf返回的是引擎级别的配置，而非特定会话的配置。

正确的做法应该是使用构造函数传入的conf参数作为会话级别的配置。这个参数才是真正代表当前会话特定配置的对象。引擎级别的配置应该用于整个引擎实例的全局设置，而会话级别的配置则用于该特定会话的个性化设置。

影响范围

这个配置层级问题可能导致以下影响：

1. 会话间配置隔离失效：不同会话可能会共享相同的配置，无法实现真正的会话隔离
2. 配置覆盖问题：会话特定的配置可能被引擎级别的配置错误覆盖
3. 预期行为偏差：开发人员期望的会话级别配置可能不会生效

解决方案

修复方案相对直接，需要修改TrinoSessionImpl类的实现，确保：

1. 使用构造函数传入的conf参数作为会话级别配置
2. 区分引擎级别配置和会话级别配置的使用场景
3. 确保会话特定的配置能够正确覆盖引擎级别的默认配置

最佳实践建议

在处理多层级配置系统时，建议：

1. 明确区分不同层级的配置来源
2. 在代码中添加清晰的注释说明配置的层级关系
3. 实现配置合并机制时考虑优先级问题
4. 为配置系统添加适当的日志记录，便于调试

这个问题虽然修复起来相对简单，但它提醒我们在设计多层级配置系统时需要特别注意各层级的边界和交互方式，确保配置能够按照预期在各个层级间正确传递和覆盖。