taos-connector-python icon indicating copy to clipboard operation
taos-connector-python copied to clipboard

[Bug] Superset Package 命名污染

Open ipaddicting opened this issue 1 year ago • 8 comments

为什么要在 taospy 里面包含 superset 的 package? Screenshot 2024-12-24 at 08 52 17

先安装 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);要么本地构建私有镜像,而不是采取这种暴力方式。

ipaddicting avatar Dec 23 '24 16:12 ipaddicting

可以加微信a15652223354 做具体排查

yu285 avatar Dec 24 '24 01:12 yu285

@ipaddicting , 我是 superset 连接驱动的开发人员,怎么方便能联系到您,或者加我微信 13811556921, 咱们方便沟通

DuanKuanJun avatar Dec 24 '24 01:12 DuanKuanJun

两位好,我的建议是:

  1. 从 taospy 中移除 superset 相关 package;
  2. 作为 taosdata 官方开发者提交 PR 至 superset 仓库合并生效

参考: Building Database Connectors for Superset Using SQLAlchemy PR: feat: add connector for CrateDB

*我这边会移除相关PR,以避免代码冲突。

ipaddicting avatar Dec 24 '24 03:12 ipaddicting

了解你的诉求,我们也正在向 apache 提交申请,希望把产品加到他们列表中,但这个需要一段时间及他们的审核

DuanKuanJun avatar Dec 24 '24 03:12 DuanKuanJun

那就最好了,因为我是在准备向 superset 提交过程中发现 package 冲突的。

麻烦稍后提供相关 PR 信息以便我跟踪后续版本更新,目前是定制化 superset 镜像部署使用的,感谢。

ipaddicting avatar Dec 24 '24 04:12 ipaddicting

好的,另外别名使用双引号我们已设置 superset 使用反引号,你说你的环境下仍然是双引号,你加我微信,详细看下

DuanKuanJun avatar Dec 24 '24 04:12 DuanKuanJun

嗯,这个改动我也看到了,所以编辑掉相关内容了...

目前我在用的版本已经手动解决掉这个问题了,后续等 superset 那边审核合并后再更新吧,谢谢。

ipaddicting avatar Dec 24 '24 05:12 ipaddicting

不客气,解决了就行

DuanKuanJun avatar Dec 24 '24 05:12 DuanKuanJun