MarigWeizhi commented on PR #3678: URL: https://github.com/apache/amoro/pull/3678#issuecomment-3139555451
> It seems this PR just rolls back the change produced by #3590. > > How can we make sure the default client is 20 now? @zhoujinsong This PR rolled back part of #3590, to avoid setting default values in two places. The default value remains 20. If the user has not set the clients, a default value will be set at [CachedHiveClientPool.java#L54](https://github.com/apache/amoro/blob/master/amoro-common/src/main/java/org/apache/amoro/hive/CachedHiveClientPool.java#L54) and default value is still 20 [CatalogMetaProperties.java#L67](https://github.com/apache/amoro/blob/master/amoro-common/src/main/java/org/apache/amoro/properties/CatalogMetaProperties.java#L67) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
