hainenber commented on code in PR #32310:
URL: https://github.com/apache/superset/pull/32310#discussion_r1962826938
##########
superset/daos/database.py:
##########
@@ -165,7 +168,30 @@ def get_ssh_tunnel(cls, database_id: int) -> SSHTunnel |
None:
)
return ssh_tunnel
-
+
+ @classmethod
+ def is_odps_partitioned_table(cls, database: Database, table_name: str):
+ if not database:
+ raise ValueError("Database not found")
+ uri = database.sqlalchemy_uri
+ passwd = database.password
+ pattern = re.compile(
+
r'odps://(?P<username>[^:]+):(?P<password>[^@]+)@(?P<project>[^/]+)/(?:\?endpoint=(?P<endpoint>[^&]+))'
Review Comment:
I mean there is no `match.group('password')` to indicate usage. Is the
naming for the captured group only for readability?
--
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-unsubscr...@superset.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org
