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

Update search indexes only when the "dynamic" setting changes from false to true/strict #1702

Merged
merged 3 commits into from
Jul 7, 2020

Conversation

Aschen
Copy link
Contributor

@Aschen Aschen commented Jul 6, 2020

What does this PR do ?

Force the search index update only if at least one dynamic field value went from false to true

@codecov
Copy link

codecov bot commented Jul 6, 2020

Codecov Report

Merging #1702 into 2-dev will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##            2-dev    #1702   +/-   ##
=======================================
  Coverage   93.53%   93.54%           
=======================================
  Files         113      114    +1     
  Lines        7212     7250   +38     
=======================================
+ Hits         6746     6782   +36     
- Misses        466      468    +2     
Impacted Files Coverage Δ
lib/service/storage/elasticsearch.js 92.40% <100.00%> (+0.17%) ⬆️
lib/kuzzle/log.js 69.23% <0.00%> (-8.55%) ⬇️
lib/util/asyncStore.js 100.00% <0.00%> (ø)
lib/api/funnel.js 96.64% <0.00%> (+0.02%) ⬆️
lib/kuzzle/kuzzle.js 95.78% <0.00%> (+0.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a85825f...8618b1d. Read the comment docs.

lib/service/storage/elasticsearch.js Outdated Show resolved Hide resolved
lib/service/storage/elasticsearch.js Outdated Show resolved Hide resolved
@Aschen Aschen force-pushed the update-search-index-when-dynamic-changes branch from bc966ec to 8618b1d Compare July 7, 2020 09:05
@sonarcloud
Copy link

sonarcloud bot commented Jul 7, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.1% 0.1% Duplication

@jenow jenow merged commit eeee47d into 2-dev Jul 7, 2020
@Aschen Aschen deleted the update-search-index-when-dynamic-changes branch July 7, 2020 11:01
@scottinet scottinet changed the title Updates search index only when dynamic changes from false to true Update search indexes only when the "dynamic" setting changes from false to true/strict Jul 7, 2020
This was referenced Jul 7, 2020
scottinet added a commit that referenced this pull request Jul 8, 2020
# [2.3.2](https://github.com/kuzzleio/kuzzle/releases/tag/2.3.2) (2020-07-07)


#### Bug fixes

- [ [#1701](#1701) ] Fix: description should not be required for plugin custom errors   ([scottinet](https://github.com/scottinet))
- [ [#1695](#1695) ] Fix dump generation   ([Aschen](https://github.com/Aschen))
- [ [#1698](#1698) ] Fix generic events on search queries   ([scottinet](https://github.com/scottinet))

#### Enhancements

- [ [#1702](#1702) ] Update search indexes only when the "dynamic" setting changes from false to true/strict   ([Aschen](https://github.com/Aschen))
- [ [#1630](#1630) ] Loose coupling: security module   ([scottinet](https://github.com/scottinet))
- [ [#1604](#1604) ] Add request ID in log   ([Aschen](https://github.com/Aschen))
---
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants