-
Notifications
You must be signed in to change notification settings - Fork 963
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
"Advanced search" returning "Page index out of bounds for available matches" error #1439
Comments
Thanks for opening your first issue here! |
Reproduced the issue on Debian 10 (Server). The issue happens with Chrome & Firefox as browsers. |
I've been comparing amundsenfrontend DEBUG logs between the newest version (defective), and an older fully functioning version (3.6.0). {
"filters": {
"database": [
"*"
]
},
"search_term": "",
"page_index": 0,
"search_type": "update_filter"
}', output='{
"search_term": "",
"msg": "Success",
"tables": {
"page_index": 0,
"results": [
{
<METADATA ABOUT MY DB>
],
"total_results": null
},
"status_code": 200
} I've noticed something very suprising in this data :
if (page_index < 0 || startIndex > total_results) {
return (
<div className="search-list-container">
<div className="search-error body-placeholder">
{PAGE_INDEX_ERROR_MESSAGE}
</div>
</div>
);
} |
which version of ES and Amundsen you are using? Might be some mismatch after introducing support for ES7 cc @verdan |
Hi,
|
@mgorsk1 I can't identify the exact origin of the problem, but I can provide logs & evidence. |
I tried using older (6) and newer versions of ES through changing my |
Also having same problem, fresh git clone, runing it with docker-compose -f docker-amundsen.yml up, executing sample_data_loader.py. (After facing same problem on #1442). Data is on Neo4j and in Elasticsearch as results shows up on direct query:
And on Neo4j: On frontend it shows up while writing search terms, like: But nothing on results search page, just "Page index out of bounds for available matches" |
I am also getting the same error as #1386 (comment) |
Hi everyone, @corridordigital : same issue for me with same version of Amundsen (Tried on Firefox and Chrome). Indeed, I noticed that the error was raised here:
In the Amundsen logs (docker compose launched in debug mode), I saw results for my advanced search. From the browser Debug Console I got :
So So this bug seems to appear because of total_results with NULL value. Thank everyone for your help on this. |
I believe this is because of the older version of Amundsen search. have you guys tried this using local docker-compose?, as I can see the version on the search package on docker hub is a couple of months older. In any way, I will release a new version of the search, that should fix this problem for you all. |
Hi @verdan 👋 |
sweet! merged.... |
Works also fine for me ! Thank your for the fix , really appreciated. |
I'm using Amundsen with neo4j and ES, with a standard docker-compose installation.
The data is ingested in neo4j (visible through browser) & Elasticsearch (visible through API calls), and I can find it and diplay it using the homepage searchbar.
Expected Behavior
Advanced Search must work and return results when there are assets that match the search keywords. This actually used to work with older versions (the one on the
master
branch, for example).Using "*" as a keyword in the Source bar should return every asset.
Current Behavior
I get a
Page index out of bounds for available matches
error everytime.Possible Solution
The error is raised by the frontend service (here).
Steps to Reproduce
Screenshots (if appropriate)
Homepage search :
Advanced search :
Context
Issue appeared with newer versions of Amundsen. It happens evey single time.
Your Environment
main
branch)The text was updated successfully, but these errors were encountered: