Skip to content

fetch gitea topics via extra request per repo#10

Open
geoheelias wants to merge 1 commit intomainfrom
feature/collect_gitea_topics
Open

fetch gitea topics via extra request per repo#10
geoheelias wants to merge 1 commit intomainfrom
feature/collect_gitea_topics

Conversation

@geoheelias
Copy link
Copy Markdown
Contributor

I left a TODO in the code, which explains the downside to this as it currently is.

@geoheelias geoheelias requested a review from puhoy July 12, 2021 18:15
@geoheelias geoheelias self-assigned this Jul 12, 2021
for repo_dict in repo_list:
# TODO unless we request topics in parallel, this slows down crawling by about 20x
# TODO if not acceptable over time, subprocess each request
repo_dict[GITEA_REPO_TOPICS_KEY] = self.crawl_repo_topics(repo_dict=repo_dict)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "exclude_topics" check allows us to control this via the indexer if we want, later on, as the state dict contain all keys from the block dict.

@puhoy puhoy changed the base branch from master to main August 13, 2021 17:35
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