[protocolbuffers/protobuf]在适当的情况下使用上下文类加载器

2024-05-13 283 views
7

修复#3474,感谢@xfxyjwf 的快速回复

ClassLoader我保留了 Android 上的现有行为,因为 grpc/grpc-java#1273 表明可能不依赖于使用上下文。我不知道有什么方法可以重用 grpc 代码,所以我只是复制了相关的部分。

我也不完全确定对 的更改java_file.cc,我没有太多扩展经验,但我认为我们希望那里有相同的行为。

我按照此处的说明进行构建,然后运行 ​​Maven 构建并且所有测试都通过了。设置单元测试来验证它是否修复了 #3474 会有点复杂,但如果您认为值得的话,我很乐意这样做。

回答

4

感谢您的拉取请求。一旦管理员之一验证此更改适合我们在我们的基础设施上运行,自动化测试就会运行。

5

感谢您的拉取请求。一旦管理员之一验证此更改适合我们在我们的基础设施上运行,自动化测试就会运行。

5

管理员之一可以验证此补丁吗?

9

嗯,我是所有提交的作者和拉取请求提交者,我不确定它指的是什么

9

可以测试

1

我合并了master,因为似乎出现了一些冲突

2

我只是想确保这份公关不会落空。我认为当前的行为显然是一个错误,并不总是有一个简单的解决方法,并且ClassNotFoundException当用户遇到这个问题时最终会感到烦恼

7

我很好奇为什么它被标记为过时和关闭?据我所知#3474仍然是一个问题

0

抱歉,我们正在批量清理和关闭已经有一段时间没有更新或我们无法采取行动的问题。如果您仍然有兴趣加入,您可以帮助更新并重新发送更改吗?如果您没有及时收到回复,请随时联系我。我们只有少数工程师致力于此存储库,并且不断失去踪迹......