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

deprecate bulk_get_timeseries_from_db? #3

Open
julffers opened this issue May 5, 2022 · 0 comments
Open

deprecate bulk_get_timeseries_from_db? #3

julffers opened this issue May 5, 2022 · 0 comments
Labels
question Further information is requested

Comments

@julffers
Copy link
Contributor

julffers commented May 5, 2022

  • pandahub version: 0.2.2
  • Python version: 3.9
  • Operating System: Windows 10

Description

I was wondering, if the Pandahub method "bulk_get_timeseries_from_db" is needed anymore. multi_get_timeseries_from_db also serves the purpose to retrieve multiple timeseries at once. The difference is, that "bulk_get_timeseries_from_db" aggregates the timeseries data directly on the database. However it is my experience, that the used MongoDB aggregations are rather complex, hard to debug and resource consuming on the database server. For most use-cases it is more efficient and less complex, to aggregate the data on the client side.

Are there any important use-cases, where it is important to use as little computing as possible client-side, that justifiy maintaining this code?

@julffers julffers added the question Further information is requested label May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant