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

[Refactor][Cherry-Pick] Synchronize OLAP external table metadata when loading data #25499

Conversation

gengjun-git
Copy link
Contributor

Currently, StarRocks use StarRocksRepository to synchronize OLAP external table metadata every 10 seconds, which is
inefficient, because the meta data will never be used when there is no load job on that external table. Change the synchronization to be triggered when there is load job on that table.

Since the synchronization is time costly, so we should release the database lock when synchronizing meta data.

backport #24739

…StarRocks#24739)

Currently, StarRocks use StarRocksRepository to synchronize OLAP
external table metadata every 10 seconds, which is
inefficient, because the meta data will never be used when there is no
load job on that external table. Change the synchronization to be
triggered when there is load job on that table.

Since the synchronization is time costly, so we should release the
database lock when synchronizing meta data.

Signed-off-by: gengjun-git <gengjun@starrocks.com>
@gengjun-git gengjun-git force-pushed the cp_olap_table_meta_sync_to_2.5 branch from 93b7fe5 to 984936a Compare June 16, 2023 13:41
@sonarcloud
Copy link

sonarcloud bot commented Jun 16, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@wanpengfei-git wanpengfei-git merged commit b42326b into StarRocks:branch-2.5 Jun 19, 2023
18 checks passed
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.

None yet

3 participants