betodealmeida commented on issue #34072:
URL: https://github.com/apache/superset/issues/34072#issuecomment-3062717809

   The other thing that could be, it looks like [we added support for catalogs 
to Doris](https://github.com/apache/superset/pull/31580). But looking at the 
code, it seems that when we don't pass a catalog (when multi-catalog is 
disabled in the database) then your URL would get rewritten to:
   
   ```
    doris://bme280:XXXXXXXXXX@pi5b:9030/internal
   ```
   
   Here's the relevant part:
   
   ```python
       @classmethod
       def adjust_engine_params(
           cls,
           uri: URL,
           connect_args: dict[str, Any],
           catalog: Optional[str] = None,
           schema: Optional[str] = None,
       ) -> tuple[URL, dict[str, Any]]:
           if catalog:
               pass
           elif uri.database and "." in uri.database:
               catalog, _ = uri.database.split(".", 1)
           else:
               catalog = "internal"
   
           # In Apache Doris, each catalog has an information_schema for BI tool
           # compatibility. See: https://github.com/apache/doris/pull/28919
           schema = schema or "information_schema"
           database = ".".join([catalog or "", schema])
           uri = uri.set(database=database)
           return uri, connect_args
   ```
   
   Since `catalog` is `None`, and you don't have a period in your database 
(specifying a schema), then the catalog gets assigned to "internal".
   
   A way to test this is to add a default schema to your URI, then it should 
point to the right catalog, can you give that a try? Something like:
   
   ```
    doris://bme280:XXXXXXXXXX@pi5b:9030/bme280.schema
   ```
   
   I'll fix the logic in the meantime, and maybe you can give it a try when my 
PR is ready?
   
   Thanks!
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to