forked from hystax/optscale
-
Notifications
You must be signed in to change notification settings - Fork 1
MPT-6785 Sync upstream/2025032501-public -> main 2025-03-31 #92
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
Merged
Conversation
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
Duplicated properties were removed from details: region, service_name, active, first_seen, last_seen
This commit introduces several changes to support retrieving clean expenses via GraphQL: - Added `qs` package for query string serialization - Created a new GraphQL query method in RestAPI client - Updated GraphQL schema to include clean expenses parameters - Modified the CleanExpensesService to use the new GraphQL query
1. Store abort controllers in the corresponding ref to associate them with specific requests rather than the query hook instance. 2. Remove the requestId parameter, as the issue with re-triggering cancellation requests no longer occurs. 3. Update loading state management to prevent UI flickering
also verified update scenario
trace log ecample
```
2025.03.06 12:21:40.723282 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Debug> MergeTask::PrepareStage: Selected MergeAlgorithm: Horizontal
2025.03.06 12:21:40.724966 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Debug> MergeTreeSequentialSource: Reading 2 marks from part 202503_1_135_27, total 1722 rows starting from the beginning of the part
2025.03.06 12:21:40.725589 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Debug> MergeTreeSequentialSource: Reading 2 marks from part 202503_136_136_0, total 28 rows starting from the beginning of the part
2025.03.06 12:21:40.726044 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Debug> MergeTreeSequentialSource: Reading 2 marks from part 202503_137_137_0, total 12 rows starting from the beginning of the part
2025.03.06 12:21:40.726559 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Debug> MergeTreeSequentialSource: Reading 2 marks from part 202503_138_138_0, total 20 rows starting from the beginning of the part
2025.03.06 12:21:40.726966 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Debug> MergeTreeSequentialSource: Reading 2 marks from part 202503_139_139_0, total 2 rows starting from the beginning of the part
2025.03.06 12:21:40.728490 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Debug> MergeTreeSequentialSource: Reading 2 marks from part 202503_140_140_0, total 2 rows starting from the beginning of the part
2025.03.06 12:21:40.752174 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Debug> MergeTask::MergeProjectionsStage: Merge sorted 1786 rows, containing 76 columns (76 merged, 0 gathered) in 0.029761612 sec., 60010.190308240024 rows/sec., 85.35 MiB/sec.
2025.03.06 12:21:40.765628 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Trace> MergedBlockOutputStream: filled checksums 202503_1_140_28 (state Temporary)
2025.03.06 12:21:40.766618 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Trace> system.query_log (9bbabecf-a2da-4bf4-bd0c-302316d04303): Renaming temporary part tmp_merge_202503_1_140_28 to 202503_1_140_28 with tid (1, 1, 00000000-0000-0000-0000-000000000000).
2025.03.06 12:21:40.766738 [ 657 ] {9bbabecf-a2da-4bf4-bd0c-302316d04303::202503_1_140_28} <Trace> system.query_log (9bbabecf-a2da-4bf4-bd0c-302316d04303) (MergerMutator): Merged 6 parts: [202503_1_135_27, 202503_140_140_0] -> 202503_1_140_28
2025.03.06 12:21:40.767141 [ 657 ] {} <Debug> MemoryTracker: Background process (mutate/merge) peak memory usage: 8.37 MiB.
```
Added 5d retention to clickhouse query_log
…le oauth and Microsoft oauth on custom deployments (#1424) ## Description The instructions 'How to configure login with Microsoft or Google authentication' are added to the documentation folder.
…s page. (#1403) ## Description OSN-650. System->Integrations description's been added.
## Description Added total_count and pagination filters into org list response Added doc file ## Related issue number OSN-737 ## Checklist * [x] The pull request title is a good summary of the changes * [ ] Unit tests for the changes exist * [ ] New and existing unit tests pass locally
## Description Validate limit, offset for organization list ## Related issue number OSN-740
fe5a6fc OSN-737. Added total_count and pagination filters into org list response 57971fd OSN-727. Add tenant id to child cloud account name on conflicting names bdb4674 OSN-650. [OptScale][Community Documentation] Add tips for Integrations page. 2ddc4d6 OSN-573. [Developers Doc] Add instruction for enable Google oauth and Microsoft oauth on custom deployments (#1424)
ffaraone
approved these changes
Apr 7, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR syncs the latest changes from upstream.