Skip to content
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

Feature/opensearch query on dual indexes #3772

Draft
wants to merge 60 commits into
base: master
Choose a base branch
from

Conversation

krharum
Copy link
Contributor

@krharum krharum commented Feb 25, 2025

No description provided.

This commit introduces a new service, DollySearchService, to enable searching and retrieving person information through Elasticsearch. It includes configurations, REST endpoints, DTO mappings, Elasticsearch queries, and related utility classes.
Introduced DTOs, enums, and repository for the Dolly Search Service to support OpenSearch operations. Added `PdlPerson` model and Elastic search consumer for indexing and query updates. Included necessary Gradle wrapper and configurations for the project build.
Deleted several unused models and associated logic to enhance code maintainability. Refactored search services to use reactive programming with Mono and improved query execution while preserving compatibility with OpenSearch.
Deleted several unused models and associated logic to enhance code maintainability. Refactored search services to use reactive programming with Mono and improved query execution while preserving compatibility with OpenSearch.
Removed dependencies and unused classes related to Elasticsearch. Disabled and commented out portions of the codebase referencing Elasticsearch components to streamline the transition. Added a new local OpenSearch configuration for testing connectivity and query-building improvements.
Adjusted OpenSearch queries to use nested structures and introduced utility methods for common query patterns. Enhanced error handling in search operations and updated data models to support additional attributes.
…ils for query building; update OpenSearchService and OpensearchController to support new search parameters
…ences; enhance query building logic in OpenSearch utilities
…eamline query execution and enhance response handling
…eamline query execution and enhance response handling #deploy-dolly-search-service
…sed properties and switch to MockitoBean for better test isolation #deploy-dolly-search-service
krharum and others added 18 commits February 19, 2025 13:00
…ckage structure and response handling #deploy-test-dolly-backend
…nse handling and improve configuration #deploy-test-dolly-backend
… handling and simplify search logic #deploy-dolly-search-service
… KrrstubClient and TpsMessaging classes #deploy-dolly-backend #deploy-test-dolly-backend
…ttings #deploy-test-dolly-search-service #deploy-dolly-search-service
@krharum krharum requested a review from a team as a code owner February 25, 2025 13:44
@krharum krharum marked this pull request as draft February 25, 2025 13:47
…environment settings #deploy-test-dolly-search-service #deploy-dolly-search-service
#deploy-test-dolly-search-service #deploy-dolly-search-service
…loy-test-dolly-search-service #deploy-dolly-search-service
…y size

- Added ElasticParamsConsumer and ElasticPutCommand for managing OpenSearch parameters
- Introduced OpensearchParameterSettings to update index settings on application startup
- Updated application.yml to include max-terms-count configuration
- Increased QUERY_SIZE in BestillingQueryService from 1000 to 10000
- Changed access level for OpenSearch in config files from read to admin #deploy-test-dolly-search-service #deploy-dolly-search-service
… and update index setting format #deploy-test-dolly-search-service #deploy-dolly-search-service
…d adjust index setting format#deploy-test-dolly-search-service #deploy-dolly-search-service
…mat#deploy-test-dolly-search-service #deploy-dolly-search-service
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.

1 participant