opensearch-project/neural-search

Refactor HybridQueryBuilderTests to decouple types from K-NN library

Offen

#979 geöffnet am 08.11.2024

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (124 Forks)auto 404
enhancementgood first issuehybrid search

Repository-Metriken

Stars
 (116 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 20h) (23 gemergte PRs in 30 T)

Beschreibung

Is your feature request related to a problem?

Coming from https://github.com/opensearch-project/neural-search/issues/859. Refactored HybridQueryTests and HybridQueryPhaseSearcherTests in https://github.com/opensearch-project/neural-search/pull/977. Similarly, need to decouple knn for HybridQueryBuilderTests.

What solution would you like?

A clear and concise description of what you want to happen.

What alternatives have you considered?

A clear and concise description of any alternative solutions or features you've considered.

Do you have any additional context?

Add any other context or screenshots about the feature request here.

Contributor Guide

Research-Richtung
Studieren Sie die Änderungen in PR #977 für HybridQueryTests und HybridQueryPhaseSearcherTests, um das Entkopplungsmuster zu verstehen. Wenden Sie dann dasselbe Muster auf HybridQueryBuilderTests an und entfernen Sie direkte Abhängigkeiten von K NN Bibliothekstypen.
Tech Stack
java
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65