Skip to content

❔ISSUE on SearchOperationsImpl #691

@SylvainAssemat

Description

@SylvainAssemat

Hi

Just to share a test that should need a litlle fix.

SearchResult result = rdFasttCompteService.query(new Query("@cap_type:{" + FASTTCOMPTE_CAP_TYPE + "} @cap_siret:{" + FASTTCOMPTE_SIRET + "}"), null);

✅ "FT.SEARCH" "fr.gms.legacy.api.redis.model.fastt.RDFasttCompteIdx" "@cap_type:{AGENCE} @cap_typecode:{809020002} @cap_siret:{91233508000020}" "DIALECT" "2"

But this

FTSearchParams params = new FTSearchParams();
params = params.limit(0, 5);

SearchResult result = rdFasttCompteService.query(new Query("@cap_type:{" + FASTTCOMPTE_CAP_TYPE + "} @cap_siret:{" + FASTTCOMPTE_SIRET + "}"), params);

❌ "FT.SEARCH" "fr.gms.legacy.api.redis.model.fastt.RDFasttCompteIdx" "redis.clients.jedis.search.Query@3dd081cc" "LIMIT" "0" "5" "DIALECT" "2"

Maybe this :

@Override
public SearchResult search(Query q, FTSearchParams params) {
   return search.ftSearch(index.toString(), q.toString(), params);
}

Should be

@Override
public SearchResult search(Query q, FTSearchParams params) {
    return search.ftSearch(index.toString(), q, params);
}

What do you think on this ?

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions