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

optimize listruns #537

Merged
merged 16 commits into from
Sep 24, 2021
Merged

optimize listruns #537

merged 16 commits into from
Sep 24, 2021

Conversation

prjemian
Copy link
Contributor

@prjemian prjemian commented Sep 23, 2021

@prjemian prjemian added this to the 1.5.2 milestone Sep 23, 2021
@prjemian prjemian self-assigned this Sep 23, 2021
@prjemian prjemian marked this pull request as draft September 23, 2021 21:32
@lgtm-com
Copy link

lgtm-com bot commented Sep 23, 2021

This pull request introduces 1 alert when merging 65afc60 into 40180d1 - view on LGTM.com

new alerts:

  • 1 for Variable defined multiple times

@lgtm-com
Copy link

lgtm-com bot commented Sep 24, 2021

This pull request introduces 1 alert when merging b6d2eb2 into 40180d1 - view on LGTM.com

new alerts:

  • 1 for Variable defined multiple times

@prjemian prjemian marked this pull request as ready for review September 24, 2021 05:01
@prjemian prjemian merged commit 5d03adc into main Sep 24, 2021
@prjemian prjemian deleted the 526-listruns branch September 24, 2021 05:19
@lgtm-com
Copy link

lgtm-com bot commented Sep 24, 2021

This pull request introduces 1 alert when merging 3ddeae3 into 40180d1 - view on LGTM.com

new alerts:

  • 1 for Variable defined multiple times

prjemian added a commit that referenced this pull request Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

listruns() is very slow listruns() -- add search by uid or scan_id
1 participant