Use alias to shorten the index names in multi-kind searchMulti kind alias
This is part of solution to fix the issue Multi kind search does not work with more than approximately 85 kinds
Type of change
-
Bug Fix -
Feature
Please provide link to gitlab issue or ADR(Architecture Decision Record)
Does this introduce a change in the core logic?
- [YES]
Does this introduce a change in the cloud provider implementation, if so which cloud?
-
AWS -
Azure -
Google Cloud -
IBM
Does this introduce a breaking change?
- [NO]
What is the current behavior?
What is the new/expected behavior?
Have you added/updated Unit Tests and Integration Tests?
Any other useful information
Merge request reports
Activity
added 1 commit
- c850054e - Refactor the codes to improve the performance to get a batch of aliases
added 1 commit
- df76724d - Optimize the performance to get the aliases for a list of kinds
added 1 commit
- 0e851ea3 - Optimize the performance to get the aliases for a list of kinds
added 1 commit
- 466cd657 - Use HashMap in memory instead of Cache as the mapping from index to alias is intact
requested review from @fhoueto.amz, @Rostislav_Dublin, @krveduru, @spencer, and @jingdongsun
@fhoueto.amz @Rostislav_Dublin @krveduru @spencer @jingdongsun Please help review this MR. The search service will use the aliases of the indices (name) instead of the index names if the list of the index names are too long (> 3840 bytes) when passing the query to the elastic search. The aliases could be created on demand if they don't exist.
@Rustam_Lotsmanenko It is fair to add integration tests though we did lots of manual tests in the dev env. The reason I didn't do is that the current integration tests (excluding my change) are not stable. I will see what I can do. Thanks.
added 31 commits
-
ae51958e...0b2b6922 - 30 commits from branch
master
- 6acab031 - Merge branch 'master' of https://community.opengroup.org/osdu/platform/system/search-service
-
ae51958e...0b2b6922 - 30 commits from branch