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.

Sign up for GitHub

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

Jump to bottom

#3466 Allow setting global SQL statement timeout for registry API & #3464 #3468

Merged
merged 14 commits into from
Jul 24, 2023
Merged

#3466 Allow setting global SQL statement timeout for registry API & #3464 #3468

merged 14 commits into from
Jul 24, 2023

Conversation

t83714
Copy link
Contributor

@t83714 t83714 commented Jun 7, 2023 •
edited
Loading

What this PR does

Checklist

  • There are unit tests to verify my changes are correct
  • I've updated CHANGES.md with what I changed.

t83714 added 12 commits June 2, 2023 20:05
@t83714
implement registry config db-query.global-timeout
36717be
@t83714
Merge remote-tracking branch 'origin/main' into issue/3466
6a68bbe
@t83714
Merge remote-tracking branch 'origin/main' into issue/3466
e21db1f
@t83714
implement db-query.default-timeout & db-query.long-query-timeout config
0becb97
@t83714
refactor the code: use the same DBsWithEnvSpecificConfig for both tes…
c8d1a65 
…t cases and non-test code
@t83714
re-write the db global timeout implementation & test cases
49ddb42
@t83714
add more test cases
9f3bd0a
@t83714
#3464 refactor registry API to allow config via .Values.appConfig
61c48ef
@t83714
#3464 fixes registry api helm chart logic
25a8413
@t83714
#3464 refactor search API & Indexer to allow config via `.Values.appC…
fa38c0f 
…onfig`
@t83714
allow to config print full config data
3247a69
@t83714
explicitly add printFullConfig to helm chart config
0060be9
@t83714 t83714 changed the title #3466 Allow setting global SQL statement timeout for registry API #3466 Allow setting global SQL statement timeout for registry API & #3464 Jul 24, 2023
@t83714
Copy link
Contributor Author

t83714 commented Jul 24, 2023

tested in dev cluster

@t83714 t83714 merged commit 7b180c1 into main Jul 24, 2023
@t83714 t83714 deleted the issue/3466 branch July 24, 2023 22:49
This was referenced Jul 24, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@t83714