tantivy #5

despiegk · 2025-08-23T03:22:05Z

despiegk commented

2025-08-23 03:22:05 +00:00

is not working, its blocking somewhere
can't use e.g. index commands

I think a specialist like lee can resolve this fast (-:

is not working, its blocking somewhere can't use e.g. index commands I think a specialist like lee can resolve this fast (-:

despiegk added 12 commits 2025-08-23 03:22:05 +00:00

... 46f96fa8cf

... fc7672c78a

... eb07386cf4

... 2743cd9c81

tanvity not working yet its blocking b68325016d

... d4d3660bac

... 7920945986

... 8bc372ea64

... 4b8216bfdb

Merge branch 'append' into tantivy ce1be0369a

* append:
  ...
  ...
  ...
  ...

# Conflicts:
#	Cargo.lock
#	Cargo.toml

Merge branch 'append' into tantivy fbcaafc86b

* append:
  ...

Merge branch 'append' into tantivy 4b3a86d73d

* append:
  ...

lee was assigned by despiegk

2025-08-23 03:24:55 +00:00

lee added 1 commit 2025-08-25 09:03:52 +00:00

Fix ft.create options parser 142084c60f

Signed-off-by: Lee Smet <lee.smet@hotmail.com>

lee added 2 commits 2025-08-25 09:16:38 +00:00

Set index writer allocated bytes to proper minimal value e9675aafed

Signed-off-by: Lee Smet <lee.smet@hotmail.com>

Format rust files ff0659b933

Signed-off-by: Lee Smet <lee.smet@hotmail.com>

lee added 1 commit 2025-08-25 10:06:20 +00:00

Fix tantivy_search_demo.sh to properly send redis commands 8decbf3375

Signed-off-by: Lee Smet <lee.smet@hotmail.com>

lee added 1 commit 2025-08-25 11:25:03 +00:00

Adapt KEYS command to actually use the pattern to search for 65c297ca94

Signed-off-by: Lee Smet <lee.smet@hotmail.com>

lee added 1 commit 2025-08-25 11:35:50 +00:00

Fix FT.SEARCH LIMIT option argument parsing e5fcaf81b5

Signed-off-by: Lee Smet <lee.smet@hotmail.com>

lee commented

2025-08-25 11:40:56 +00:00

Fixed issues in code so both examples (tantivity_search_demo.sh and test_tantivity_integration.sh) run to completion. Basically:

Fixed parsing of options for FT.CREATE command
Set the allocated memory for the index writer to an acceptable value
Modified the way tantivy_search_demo.sh sends redis commands to work around bash escaping ' symbols
Fixed parsing of the LIMIT option for the FT.SEARCH command
Modified the KEYS command to actually account for the pattern to search for instead of requiring the static * search string

Note that the KEYS command (which appears to do text search) is not using tantivy right now and instead hits the storage backend directly (I didn't change the behavior), which does not seem to be the intended result.

Fixed issues in code so both examples (tantivity_search_demo.sh and test_tantivity_integration.sh) run to completion. Basically: - Fixed parsing of options for FT.CREATE command - Set the allocated memory for the index writer to an acceptable value - Modified the way tantivy_search_demo.sh sends redis commands to work around bash escaping `'` symbols - Fixed parsing of the LIMIT option for the FT.SEARCH command - Modified the KEYS command to actually account for the pattern to search for instead of requiring the static `*` search string Note that the KEYS command (which appears to do text search) is not using tantivy right now and instead hits the storage backend directly (I didn't change the behavior), which does not seem to be the intended result.

maximevanhees changed target branch from append to main

2025-09-11 13:35:14 +00:00

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin tantivy:tantivy

git checkout tantivy

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: herocode/herodb#5