Skip to content

Conversation

patrick-werner
Copy link
Member

This pull request introduces a new configuration property to control whether resources are marked for reindexing when search parameters change. The property is added to AppProperties, with corresponding getter and setter methods, and is wired through to JpaStorageSettings in the FHIR server configuration.

Configuration enhancements:

  • Added a new Boolean property mark_resources_for_reindexing_upon_search_parameter_change to the AppProperties class, defaulting to true.
  • Implemented getter and setter methods for the new property in AppProperties.

Integration with FHIR server settings:

  • Updated FhirServerConfigCommon to pass the new property from AppProperties to JpaStorageSettings via the setMarkResourcesForReindexingUponSearchParameterChange method.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new configuration property to control whether resources are marked for reindexing when search parameters change. It enhances the existing configuration system by allowing administrators to disable automatic reindexing behavior.

  • Added a new boolean configuration property with getter/setter methods
  • Integrated the property with JPA storage settings to control reindexing behavior

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
AppProperties.java Added new boolean property with default value true and corresponding getter/setter methods
FhirServerConfigCommon.java Wired the new property from AppProperties to JpaStorageSettings

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jkiddo
Copy link
Collaborator

jkiddo commented Oct 9, 2025

@patrick-werner I'll approve once your spotless checks out

@patrick-werner
Copy link
Member Author

@patrick-werner I'll approve once your spotless checks out

@jkiddo done

@jkiddo
Copy link
Collaborator

jkiddo commented Oct 9, 2025

@patrick-werner conflicts now - because of your other PR 🙈

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