Skip to content

[feat][COM][httpclient] Optimize BML client connection pool parameters and fix datasource client issues#963

Closed
v-kkhuang wants to merge 1 commit intodev-1.9.0from
relaese-1.9.0-bml-http-conect
Closed

[feat][COM][httpclient] Optimize BML client connection pool parameters and fix datasource client issues#963
v-kkhuang wants to merge 1 commit intodev-1.9.0from
relaese-1.9.0-bml-http-conect

Conversation

@v-kkhuang
Copy link
Copy Markdown

$(cat <<'EOF'

Summary

This PR includes the following main changes:

1. BML Client Connection Pool Optimization (linkis-commons/linkis-httpclient)

  • Optimized connection pool configuration in AbstractHttpClient.scala
  • Set connection pool parameters explicitly via PoolingHttpClientConnectionManager
  • Added pending connection count monitoring and logging
  • Improved connection management for better performance and reliability

2. DataSource Client Cleanup (linkis-public-enhancements/linkis-pes-client)

  • Removed deprecated getPublishedDataSourceByType method
  • Simplified LinkisDataSourceRemoteClient interface

3. Other Changes

  • Fixed compilation issues across multiple modules
  • Updated HBase engineconn plugin resources
  • Updated dependencies and build configurations
  • Frontend and backend dependency updates

Related Issues

None

Test Plan

  • Code compiles successfully
  • Unit tests pass
  • Integration tests pass

Code Change Details

linkis-commons/linkis-httpclient/src/main/scala/org/apache/linkis/httpclient/AbstractHttpClient.scala

  • Added explicit connection pool configuration before building HttpClient
  • Moved setMaxTotal and setDefaultMaxPerRoute to connectionManager
  • Added pending connection count to connection stats logging

linkis-public-enhancements/linkis-pes-client/src/main/scala/org/apache/linkis/datasource/client/impl/LinkisDataSourceRemoteClient.scala

  • Removed getPublishedDataSourceByType method implementation

🤖 Generated with Claude Code
EOF
)

@v-kkhuang v-kkhuang closed this Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant