[NAE-2060] Merge 6.2.10, 6.4.2 into 6.5.0#439
Open
SamuelPalaj wants to merge 112 commits into
Open
Conversation
- add property to component on dashboard
…Count error - ElasticCaseService, ElasticViewPermissionService: refactored queries to use termsQuery and filter, reducing clause count. - pom.xml: updated spring-session to spring-session-core.
…Count error - ElasticCaseService: incorporation of reminders from PR
[NAE-2053] Optimize ElasticCaseService queries to eliminate maxClauseCount error
[NAE-2031] Dashboard bug fix
- update CHANGELOG.md
- update changelog and version
- update release date in changelog
- implement exporting cases from case view - added ExportController, XlsExportService, XlsExportConfiguration, XlsExportDateUtils - added attribute allow export to preference_item and MenuItemBody - extend IElasticCaseService with existing method to avoid code duplication - add simple test
- resolve comments - add documentation and validations for xls export properties - update error messages - update controller mapping - update bean initialization in XlsExportService - update test
- update version - add changelog
[NAE-2100] Case view export button as NAE feature
…f bulk indexes set
…ns in bulk requests
- **Add bulk reindexing support**:
- Introduced the `bulkIndex` method in `IElasticIndexService`, enabling reindexing of all or stale cases and tasks.
- Created a dedicated `IndexParams` class to encapsulate parameters for the reindexing process (e.g., `indexAll`, `caseBatchSize`, `taskBatchSize`).
- Optimized the index batching process for cases and tasks by leveraging configurable batch sizes.
- **Remove deprecated methods**:
- Removed the old `bulkIndex` method from `IElasticIndexService`.
- Eliminated the obsolete `IBulkService` interface.
- **Update ElasticController**:
- Changed endpoint, `/reindex/bulk`, for initiating bulk reindexing with configurable parameters.
- Updated controller logic to leverage the new `bulkIndex` method for improved performance and scalability.
- **Configuration enhancements**:
- Extended `ElasticsearchProperties` to include `IndexProperties`, allowing batch sizes for cases and tasks to be configured via properties.
- **Code cleanup**:
- Removed unused imports and annotated services with `@RequiredArgsConstructor`.
- Streamlined reindexing tasks by integrating advanced filtering and improved logging.
This commit enhances the maintainability and scalability of Elasticsearch reindexing while introducing configurability and improved documentation for reindexing processes.
- updated request param
Added detailed Javadoc comments to improve the clarity of key methods and fields in `ElasticIndexService` and `IndexParams`. This documentation provides insights into parameter usage, default values, and functionalities, facilitating better understanding and maintenance.
Enable and configure JitPack repository for dependency resolution. Update QRGen dependency to version 3.0.1 with updated groupId and artifactId for compatibility.
Updated the reindexing logic to replace BooleanExpression predicates with MongoDB query objects, improving clarity and alignment with MongoDB operations. Adjusted method signatures and internal calls to support the new query-based approach. Renamed `bulkIndex` to `bulkReindex` for consistency with functionality.
Removed unused parameters `now`, `after`, and `indexAll` from the `reindexQueried` method and its invocation. This streamlines the method signature and improves clarity by reducing unnecessary complexity.
…utdown. - Add `existsById` method and improve task resolution - Introduced a new `existsById` method in `IUserService` to check user existence. - Modified `resolveUserRef` in `TaskService` to return a list of resolved tasks instead of void. - Ensured consistency in validations by replacing `size()` checks with `isEmpty()`.
- Introduced `MigrationProperties` class to define configurable properties for migration, including skip list, cache eviction, and shutdown control. - Enhanced `MigrationOrderedCommandLineRunner` to respect `MigrationProperties` settings. - Improved logging for migration operations with additional conditions.
…esses to flush work after migration.
- Upgraded `actions/checkout` from v3 to v6 across workflows for feature improvements and compatibility. - Updated `actions/setup-java` from v3 to v5 and switched JDK distribution to `temurin`, adding Maven caching where applicable. - Upgraded MongoDB image to `mongo:6` and Elasticsearch image to `elasticsearch:7.17.28` in workflows. - Removed deprecated Maven cache setup and unused steps to simplify workflows.
- Added new Maven repositories for central and snapshot artifacts in `pom.xml`. - Switched JDK distribution to `temurin` and enabled Maven caching in GitHub workflows. - Upgraded MongoDB service to `mongo:6` and Elasticsearch to `elasticsearch:7.17.28`. - Updated dependencies including `xml-apis-ext` and various GitHub Actions versions for compatibility.
- Introduced `UserServiceTest` to validate user existence and ID checks. - Enhanced `ApplicationShutdownProvider` to properly shut down thread pool executors for graceful application termination.
Minor improvements to manage migrations
- Add endpoint to reload tasks of all cases
- add lastModifiedDataSet property to Case and transient property changed to DataField, which is set when value, choices or options on DataField are set
- change changelog and version
- fix duplicity
[NAE-2101] Release 6.4.2
- changes according to PR
- changes according to PR
- add task to changelog
[NAE-2401] Timestamp of case dataSet change
- fix vulnerability
- fix null pointer
- fix null pointer
[NAE-2101] Release 6.4.2
# Conflicts: # CHANGELOG.md # docker-compose.yml # pom.xml # src/main/java/com/netgrif/application/engine/elastic/service/ElasticCaseService.java
# Conflicts: # .github/workflows/master-build.yml # .github/workflows/pr-build.yml # .github/workflows/release-build.yml # CHANGELOG.md # docker-compose.yml # pom.xml # src/main/java/com/netgrif/application/engine/elastic/service/ElasticCaseService.java # src/main/java/com/netgrif/application/engine/workflow/domain/menu/MenuItemBody.java # src/main/java/com/netgrif/application/engine/workflow/domain/menu/MenuItemConstants.java # src/main/resources/petriNets/engine-processes/preference_item.xml
- after merge fixes
|
Important Review skippedToo many files! This PR contains 257 files, which is 107 over the limit of 150. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (257)
You can disable this status message by setting the Use the checkbox below for a quick retry:
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
mazarijuraj
approved these changes
May 11, 2026
Contributor
mazarijuraj
left a comment
mazarijuraj
left a comment
There was a problem hiding this comment.
Can be merged, 💩 will be cleaned up during Core merge
| // log.warn(String.format("DateFields should have LocalDate values! DateField (%s) with Date value found! Value will be converted for indexation.", netField.getImportId())); | ||
| LocalDateTime transformed = this.transformDateValueField(dateField); | ||
| return formatDateField(LocalDateTime.of(transformed.toLocalDate(), LocalTime.NOON)); | ||
| return formatDateField(LocalDateTime.of(transformed.toLocalDate(), LocalTime.MIDNIGHT)); |
| return null; | ||
| else | ||
| return query; | ||
| return resultAlwaysEmpty ? null : query; |
| processQuery.setGroup(request.group); | ||
| List<PetriNetReference> groupProcesses = this.petriNetService.search(processQuery, user, new FullPageRequest(), locale).getContent(); | ||
| if (groupProcesses.size() == 0) | ||
| if (groupProcesses.isEmpty()) { |
Contributor
There was a problem hiding this comment.
This check should be done in a separate function and first in buildSingleQuery
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
merge of versions 6.2.10 and 6.4.2 into 6.5.0
Implements NAE-2060
How Has Been This Tested?
Automated tests for version 6.5.0 were run on this branch
Test Configuration
Checklist: