Skip to content

Conversation

@sunmiaozju
Copy link
Collaborator

No description provided.

@sunmiaozju sunmiaozju requested a review from LLLLKKKK as a code owner February 8, 2026 06:06
@sunmiaozju sunmiaozju force-pushed the feature/update_frontend_master_client branch from f461a86 to 49352cf Compare February 8, 2026 06:08
help="Master queue reject threshold",
)

master_group.add_argument(
Copy link
Collaborator

Choose a reason for hiding this comment

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

--flexlb_xxxx

route_logger.warning(
f"FlexLb cached queue length {queue_length} exceeds threshold "
f"{StaticConfig.master_config.master_queue_reject_threshold}, "
f"proactively rejecting request <{input.request_id}>"
Copy link
Collaborator

Choose a reason for hiding this comment

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

加个qps metrics

"""获取或创建HTTP session"""
if self._session is None or self._session.closed:
timeout = ClientTimeout(total=0.5)
timeout = ClientTimeout(total=3600)
Copy link
Collaborator

Choose a reason for hiding this comment

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

顺便给他加个配置?默认值:如果开启queue 模式是3600; 不开是0.5 ;设置了以设置为准

def __init__(self, max_connect_pool_size=1000):
self.max_connect_pool_size = max_connect_pool_size
def __init__(self, host_service=None, server_config=None, master_config=None):
frontend_server_count = server_config.frontend_server_count if server_config else 4
Copy link
Collaborator

Choose a reason for hiding this comment

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

self.max_connect_pool_size 直接从master_config.master_max_connect_pool_size 里面获取 不搞这个奇怪逻辑



@dataclass
class HostHeartbeatInfo:
Copy link
Collaborator

Choose a reason for hiding this comment

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

换个名字 FlexlbHeartbeatInfo

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.

2 participants