[Bug] Superset Package 命名污染
为什么要在 taospy 里面包含 superset 的 package?
先安装 taospy 2.7.18后,该代码提交会导致 superset package 出现命名冲突,无法通过执行 pip install -e . 来构建 superset 本地开发环境。
初始化 superset 本地开发环境后,再安装 taospy 2.7.18 同样会因为 package 冲突导致 superset cli 失效。
报错如下:
Traceback (most recent call last):
File "/home/vscode/.local/bin/superset", line 5, in <module>
from superset.cli.main import superset
File "/workspaces/superset/superset/cli/main.py", line 27, in <module>
from superset import app, appbuilder, cli, security_manager
ImportError: cannot import name 'appbuilder' from 'superset' (unknown location)
package 信息如下:
vscode ➜ /workspaces/superset (feat-tdengine) $ ls /usr/local/lib/python3.10/site-packages/superset
db_engine_specs
vscode ➜ /workspaces/superset (feat-tdengine) $ ls /usr/local/lib/python3.10/site-packages/superset/db_engine_specs
TDengine.py __pycache__
作为数据库连接器 taospy 不应该擅自覆盖除 taos/taos* 以外其他库或软件的 pacakge。
如果需要在 superset 中使用 tdengine 要么提交官方 PR 合并代码(我正在尝试完善PR);要么本地构建私有镜像,而不是采取这种暴力方式。
可以加微信a15652223354 做具体排查
@ipaddicting , 我是 superset 连接驱动的开发人员,怎么方便能联系到您,或者加我微信 13811556921, 咱们方便沟通
两位好,我的建议是:
- 从 taospy 中移除 superset 相关 package;
- 作为 taosdata 官方开发者提交 PR 至 superset 仓库合并生效
参考: Building Database Connectors for Superset Using SQLAlchemy PR: feat: add connector for CrateDB
*我这边会移除相关PR,以避免代码冲突。
了解你的诉求,我们也正在向 apache 提交申请,希望把产品加到他们列表中,但这个需要一段时间及他们的审核
那就最好了,因为我是在准备向 superset 提交过程中发现 package 冲突的。
麻烦稍后提供相关 PR 信息以便我跟踪后续版本更新,目前是定制化 superset 镜像部署使用的,感谢。
好的,另外别名使用双引号我们已设置 superset 使用反引号,你说你的环境下仍然是双引号,你加我微信,详细看下
嗯,这个改动我也看到了,所以编辑掉相关内容了...
目前我在用的版本已经手动解决掉这个问题了,后续等 superset 那边审核合并后再更新吧,谢谢。
不客气,解决了就行