You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think that the current, multi-apps architecture (xpublish's FastAPI apps + Flask app) will quickly become problematic if we want to add more realistic examples. Currently:
For each dataset, we need to run a xpublish FastAPI application
Additionally, we need to run a top-level FastAPI application, acting like a reverse proxy, in which we mount all single dataset applications
If we want to add or remove one or more datasets, we need to manage 1 and 2 manually or automatically and restart the top-level application.
The Flask application mirrors some API provided by xpublish (duplicated maintenance)
Accessing the datasets in the Flask application using fsspec's HTTPFileSystem introduces overhead.
I'm not sure if there is any benefit of having all those layers here. It's probably not worth the extra complexity.
Running a single FastAPI application would be much simpler, IMO. That's not currently possible with xpublish, but it could be supported with little effort (see https://github.com/jhamman/xpublish/issues/25).
We should add a more realistic example here.
The text was updated successfully, but these errors were encountered: