Skip to content

use Docker to build pisa, update results#89

Open
PSeitz wants to merge 1 commit into
masterfrom
tantivy_main
Open

use Docker to build pisa, update results#89
PSeitz wants to merge 1 commit into
masterfrom
tantivy_main

Conversation

@PSeitz

@PSeitz PSeitz commented May 31, 2026

Copy link
Copy Markdown
Collaborator

update Makefile to use lucene 10.4
update results.json
use Docker to build pisa
use Docker to run on mac for serenedb and pisa (untested)
update tantivy-main hash

update Makefile to use lucene 10.4
update results.json
use Docker to build pisa
use Docker to run on mac for serenedb and pisa (untested)

@jpountz jpountz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for updating the Lucene version in the Makefile and doing a new run. Did you use a c7i.2xlarge as advertised in the README?

@PSeitz

PSeitz commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator Author

I used 9800X3D with 32GB RAM

@jpountz

jpountz commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator

@jpountz

jpountz commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

From a quick look at the results, PISA seems to parse negated terms as non-existing terms, which in turn makes it report too good latencies.

@jpountz

jpountz commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

@PSeitz any reason not to merge this PR? I'm keen to get the updated (better) results for Lucene 10.4 published. I think we can deal with the PISA problem later on?

@PSeitz

PSeitz commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator Author

Nice CPU. :) Let's update the README then? https://github.com/quickwit-oss/search-benchmark-game/blob/master/README.md?plain=1#L58-L63

My most prized possession :)
I don't have access to my computer currently. Will update it next week

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