Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Superset Package 命名污染 #288

Open
ipaddicting opened this issue Dec 23, 2024 · 8 comments
Open

[Bug] Superset Package 命名污染 #288

ipaddicting opened this issue Dec 23, 2024 · 8 comments

Comments

@ipaddicting
Copy link
Contributor

ipaddicting commented Dec 23, 2024

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

@yu285
Copy link

yu285 commented Dec 24, 2024

可以加微信a15652223354 做具体排查

@DuanKuanJun
Copy link
Contributor

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

@ipaddicting
Copy link
Contributor Author

两位好,我的建议是:

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

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

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

@DuanKuanJun
Copy link
Contributor

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

@ipaddicting
Copy link
Contributor Author

ipaddicting commented Dec 24, 2024

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

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

@DuanKuanJun
Copy link
Contributor

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

@ipaddicting
Copy link
Contributor Author

ipaddicting commented Dec 24, 2024

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

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

@DuanKuanJun
Copy link
Contributor

不客气,解决了就行

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants