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

Auto index creation fixes #1926

Merged
merged 8 commits into from
Apr 24, 2024
Merged

Auto index creation fixes #1926

merged 8 commits into from
Apr 24, 2024

Conversation

gvolfing
Copy link
Contributor

@gvolfing gvolfing commented Apr 10, 2024

The auto index creation is experiencing issues with transactions that create more than one entities at a time. This fix is resolving that issue.

@gvolfing gvolfing self-assigned this Apr 10, 2024
@gvolfing gvolfing added bug bug Docs unnecessary Docs unnecessary labels Apr 11, 2024
@gvolfing gvolfing added this to the mg-v2.17.0 milestone Apr 11, 2024
@hal-eisen-MG hal-eisen-MG modified the milestones: mg-v2.17.0, mg-v2.16.1 Apr 15, 2024
@gvolfing gvolfing added Docs - changelog only Docs - changelog only and removed Docs unnecessary Docs unnecessary labels Apr 15, 2024
@gvolfing gvolfing added this pull request to the merge queue Apr 22, 2024
Copy link

sonarcloud bot commented Apr 22, 2024

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 22, 2024
@gvolfing gvolfing added this pull request to the merge queue Apr 22, 2024
@gvolfing gvolfing removed this pull request from the merge queue due to a manual request Apr 22, 2024
@gvolfing gvolfing added this pull request to the merge queue Apr 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 22, 2024
@gvolfing gvolfing added this pull request to the merge queue Apr 24, 2024
Merged via the queue into master with commit fc890ac Apr 24, 2024
10 checks passed
@gvolfing gvolfing deleted the auto-index-creation-fixes branch April 24, 2024 09:56
@kgolubic
Copy link
Contributor

@gvolfing for RN I would use:

Resolved an issue in the automatic index creation process where transactions creating multiple entities simultaneously could lead to unexpected behaviors. This patch ensures reliable auto index creation when dealing with multiple entity creations in a single transaction.

Is that OK?

@gvolfing
Copy link
Contributor Author

@kgolubic Perfect.

MarkoBarisic pushed a commit that referenced this pull request Apr 25, 2024
* Fixes

* Include missing header

* Add messages for debuging

* Clang-tidy

* Remove unnecessary logging
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bug Docs - changelog only Docs - changelog only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants