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

Adjust DBS Client to aggregate data from dbs1go server #658

Closed
vkuznet opened this issue Oct 11, 2021 · 5 comments
Closed

Adjust DBS Client to aggregate data from dbs1go server #658

vkuznet opened this issue Oct 11, 2021 · 5 comments

Comments

@vkuznet
Copy link
Contributor

vkuznet commented Oct 11, 2021

Upon discussion within DMWM we agreed to adjust DBS Client to aggregate data from dbs2go server. This issue is a placeholder with requirements, which are the following:

  • add aggregation functionality to sub-set of DBS APIs, e.g. runs, etc., see full discussion in Difference in DBS python vs dbs2go result output WMCore#10841
  • add additional option to perform aggregation, and make it true by default. This will allow later to change the default to false (i.e. do not perform aggregation) and make it transparent to upstream users of DBS client
  • ensure that DBS Client will work transparently with existing Python server and new dbs2go one
  • once all changes will be place make new release and install it with CMS software stack (WMA, CRAB, etc.) such that DBS Client will be ready for migration from Python to Go based DBS srever
@vkuznet
Copy link
Contributor Author

vkuznet commented Oct 11, 2021

@klannon , @amaltaro , @yuyiguo this is a placeholder ticket to list set of requirements we discussed will be required to complete migration to Go based DBS server. Please review and add/change necessary items.

@vkuznet
Copy link
Contributor Author

vkuznet commented Oct 11, 2021

for the record, I'll do all the changes and provide appropriate PR to satisfy listed set of requirements.

@vkuznet
Copy link
Contributor Author

vkuznet commented Oct 11, 2021

@amaltaro , @yuyiguo please let me know which branch I should use to start the development.

@klannon
Copy link

klannon commented Oct 11, 2021

I may be wrong, but I think we'd want to use the new client only repository for this, wouldn't we?

@vkuznet
Copy link
Contributor Author

vkuznet commented Oct 12, 2021

ok, closing this one in favor of dmwm/DBSClient#19

@vkuznet vkuznet closed this as completed Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants