Skip to content

[Enhancement](multi-catalog) expose config: hms client pool size.#21321

Closed
dutyu wants to merge 1 commit intoapache:masterfrom
dutyu:expose-hms-client-poolsize
Closed

[Enhancement](multi-catalog) expose config: hms client pool size.#21321
dutyu wants to merge 1 commit intoapache:masterfrom
dutyu:expose-hms-client-poolsize

Conversation

@dutyu
Copy link
Contributor

@dutyu dutyu commented Jun 29, 2023

Proposed changes

Sometimes the default hms client pool size is not enough, so we need expose this config.
This pr do the following changes:

  1. add a config which can set the hms client pool size
  2. fix the spelling mistake for max_external_file_catch_num

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@dutyu dutyu marked this pull request as ready for review June 29, 2023 06:50
@dutyu
Copy link
Contributor Author

dutyu commented Jun 29, 2023

run buildall

@morningman morningman added the dev/2.0.0 2.0.0 release label Jun 29, 2023
Copy link
Contributor

@Jibing-Li Jibing-Li left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@dutyu dutyu closed this Jul 3, 2023
@morningman morningman removed the dev/2.0.0 2.0.0 release label Jul 3, 2023
@dutyu dutyu deleted the expose-hms-client-poolsize branch July 9, 2023 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants