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

Remote search does not yield any results #7893

Closed
xoriole opened this issue Feb 8, 2024 · 2 comments · Fixed by #7896
Closed

Remote search does not yield any results #7893

xoriole opened this issue Feb 8, 2024 · 2 comments · Fixed by #7896
Assignees
Milestone

Comments

@xoriole
Copy link
Contributor

xoriole commented Feb 8, 2024

Running the current main from source with an empty state directory does not show any search results.

In the beginning, there are no peers so it is expected that there are no results.
However, even after the peers are discovered, no results from the remote peers are available.

The recent refactoring of PopularityCommunity and RemoteQueryCommunity to ContentDiscoveryCommunity kept the same community ID, so the peers should be discoverable and the results should be available. But, it is not the case which seems like a bug.

To reproduce, the local search needs to be disabled manually from the code (one way is to return empty [] results from /search/local present in database_endpoints.py).

The issue seems platform-independent but tested on Ubuntu.

@xoriole xoriole added this to the 7.14.0 milestone Feb 8, 2024
@xoriole xoriole self-assigned this Feb 8, 2024
@synctext
Copy link
Member

synctext commented Feb 9, 2024

Wow, that is a very basic feature!
Is it helpful to make a test of this to reproduce? Either with mocked remote peer or something else.

@xoriole
Copy link
Contributor Author

xoriole commented Feb 9, 2024

I can reproduce the issue in a test and have a fix in this PR: #7896.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants