You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by "Smallhi (via GitHub)" <gi...@apache.org> on 2024/04/20 08:04:16 UTC
[I] Compatibility Issues with Aliyun MaxCompute after Upgrading Superset to 4.0 [superset]
Smallhi opened a new issue, #28160:
URL: https://github.com/apache/superset/issues/28160
### Bug description
We are currently utilizing Aliyun MaxCompute as our database specification. However, after our recent upgrade to Superset version 4.0, we have encountered several issues.
Superset appears to be incompatible with tables formatted as 'sales.dwd_user_action'. The existing datasets cannot be reached through Superset. However, by manually modifying the database entry to 'dwd_user_action', the table becomes accessible.
When attempting to create a dataset for the 'dwd_user_action' table, which contains a large number of columns, Superset fails with the following error message:
"The SQL is invalid and cannot be parsed. Unable to load columns for the selected table. Please select a different table."
Due to this being a production environment, I have temporarily made changes to the code in helpers.py, but this is not a sustainable solution.
We request assistance in resolving these compatibility issues. Your help would be greatly appreciated.
Thank you for your attention to this matter.
```python
def get_query_str_extended(
self,
query_obj: QueryObjectDict,
mutate: bool = True,
) -> QueryStringExtended:
sqlaq = self.get_sqla_query(**query_obj)
sql = self.database.compile_sqla_query(sqlaq.sqla_query)
sql = self._apply_cte(sql, sqlaq.cte)
try:
if 'maxcompute' == self.db_engine_spec.engine:
sql = sqlparse.format(sql, reindent=True)
else:
sql = SQLStatement(sql, engine=self.db_engine_spec.engine).format()
except SupersetParseError:
logger.warning("Unable to parse SQL to format it, passing it as-is"
```
### How to reproduce the bug
1. use odps datasource
2. import dataset
3.
### Screenshots/recordings
_No response_
### Superset version
4.0.0
### Python version
3.9
### Node version
16
### Browser
Chrome
### Additional context
_No response_
### Checklist
- [ ] I have searched Superset docs and Slack and didn't find a solution to my problem.
- [ ] I have searched the GitHub issue tracker and didn't find a similar bug report.
- [ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.
--
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: notifications-unsubscribe@superset.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Compatibility Issues with Aliyun MaxCompute after Upgrading Superset to 4.0 [superset]
Posted by "zhaoyongjie (via GitHub)" <gi...@apache.org>.
zhaoyongjie commented on issue #28160:
URL: https://github.com/apache/superset/issues/28160#issuecomment-2120454434
@TimothyDing Thanks for the mention, I'll check it out and reply you guys after finding any clues.
--
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: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Compatibility Issues with Aliyun MaxCompute after Upgrading Superset to 4.0 [superset]
Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on issue #28160:
URL: https://github.com/apache/superset/issues/28160#issuecomment-3156072677
We haven't heard back, and we have no means of testing, so I'll assume the new release resolved things. If not, say the word, and we can reopen this. Thank you for reporting it, and thanks for the help/pointers!
--
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: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Compatibility Issues with Aliyun MaxCompute after Upgrading Superset to 4.0 [superset]
Posted by "TimothyDing (via GitHub)" <gi...@apache.org>.
TimothyDing commented on issue #28160:
URL: https://github.com/apache/superset/issues/28160#issuecomment-2120420299
@zhaoyongjie The submitter is a senior developer at Alibaba who uses Superset, and this PR is very important for promoting Superset to MaxCompute users.Please have a look.
--
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: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Compatibility Issues with Aliyun MaxCompute after Upgrading Superset to 4.0 [superset]
Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on issue #28160:
URL: https://github.com/apache/superset/issues/28160#issuecomment-2840348777
Is this still an issue, or did the linked PyODPS fix solve things?
--
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: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Compatibility Issues with Aliyun MaxCompute after Upgrading Superset to 4.0 [superset]
Posted by "wjsi (via GitHub)" <gi...@apache.org>.
wjsi commented on issue #28160:
URL: https://github.com/apache/superset/issues/28160#issuecomment-2568625918
@TimothyDing @Smallhi Should be resolved with PyODPS 0.12.2 release. See https://github.com/aliyun/aliyun-odps-python-sdk/releases/tag/v0.12.2 for details.
--
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: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [I] Compatibility Issues with Aliyun MaxCompute after Upgrading Superset to 4.0 [superset]
Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas closed issue #28160: Compatibility Issues with Aliyun MaxCompute after Upgrading Superset to 4.0
URL: https://github.com/apache/superset/issues/28160
--
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: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org