aisk commented on PR #8420: URL: https://github.com/apache/paimon/pull/8420#issuecomment-4881826815
> @aisk What is the purpose of creating this PR? I see that you banned 3.6 again before. Do you really want to improve 3.6 or do something about it? The purpose is at the original PR description, and this is the Chinese translation: 对于 Python 3.6/3.7，`requirements.txt` 中指定的 `pyroaring` 版本并不包含 `BitMap64`。因此，在这些 Python 版本下，导入或使用会创建 `RoaringBitmap64` 的 PyPaimon 代码路径时可能会失败。要求使用 `pyroaring 1.0.0+` 可以提供 `BitMap64`，但这些版本没有发布适用于 Python 3.6/3.7 的 wheel，因此用户需要从源码构建 `pyroaring`。 根据 @JingsongLi 的要求，取消 PyPaimon 对 Python3.6 / 3.7 的支持，或者直接要求这两个版本的用户需要本地安装 GCC 等编译器而在本地构建 pyroaring 都被否定了，因此只能通过在 3.6 / 3.7 只下创建 fallback 来解决这个问题。 本 PR 为 `pyroaring.BitMap64` 不可用的环境增加了一个回退实现。`RoaringBitmap64` 在 `BitMap64` 可用时仍会使用它；否则，会按照高 32 位对数值进行分块，并使用普通的 32 位 `pyroaring.BitMap` 存储每个分块。依赖版本范围和 CI 安装逻辑也进行了相应调整：Python 3.6/3.7 使用已经发布 wheel 的 `pyroaring` 版本，而 Python 3.8+ 继续使用 `pyroaring>=1.0.0`。 Can I re-open this PR now? 我能重新打开这个 PR 吗？ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
