Skip to content

Follow up corrections for issue #382#387

Merged
encukou merged 5 commits intofedora-python:masterfrom
irushchyshyn:follow-ups
Mar 9, 2018
Merged

Follow up corrections for issue #382#387
encukou merged 5 commits intofedora-python:masterfrom
irushchyshyn:follow-ups

Conversation

@irushchyshyn
Copy link
Copy Markdown
Contributor

@irushchyshyn irushchyshyn commented Mar 8, 2018

  • On index page subqueryload only run time requirements (as it was before);
  • Do a bit of refactoring for Py3Query:
    Skip the step of querying ambiguous requires, as after we started collecting all build and run time requirers of python packages, we already have the data needed.
  • Add legacy-leaf status for history data collection;
  • Do Fedora update (although there were no composes) to add legacy-leaf status to history.csv and make sure the query refactoring did not break anything.

#382

After we started collecting build and run time requirers of python
packages, we no longer need to run another query to get ambiguous
requirers. We can just use the data we already have, and it speeds up
the data collection.
@irushchyshyn irushchyshyn requested a review from encukou March 8, 2018 17:49
@irushchyshyn irushchyshyn changed the title Follow up corrections for #382 Follow up corrections for issue #382 Mar 8, 2018
@encukou encukou merged commit a2f3b61 into fedora-python:master Mar 9, 2018
@encukou
Copy link
Copy Markdown
Member

encukou commented Mar 9, 2018

Thanks!

@irushchyshyn irushchyshyn deleted the follow-ups branch March 15, 2018 10:11
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

Successfully merging this pull request may close these issues.

2 participants