[python] Register fitable to the registration center during heartbeat#217
Merged
CodeCasterX merged 1 commit intoModelEngine-Group:3.5.xfrom Jul 23, 2025
Merged
Conversation
Member
|
该修改似乎和注册中心的逻辑有关,当前使用老的注册中心,可能是注册中心的不完善导致了该现象的出现,目前正在推进使用最新的Nacos来作为注册中心的最新实现,我建议,当前的优化仅为临时优化,待Nacos上线后,修改回来,验证是否还存在当前问题。 #178 |
9aaefb8 to
f09c2c5
Compare
CodeCasterX
approved these changes
Jul 23, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 相关问题 / Related Issue
原先逻辑是在心跳重新连接并恢复后注册 fitable 信息到注册中心,但是在验证的过程中,还是会发现因为网络或者注册中心未及时启动导致 python 进程注册失败,导致未成功注册信息到注册中心,因此修改为每次心跳注册 fitable 信息到注册中心。 / The original logic was to register the fitable information to the registration center after the heartbeat was reconnected and restored. However, during the verification process, it was found that the python process registration failed because the network or the registration center was not started in time, resulting in unsuccessful registration of the information to the registration center. Therefore, it was changed to register the fitable information to the registration center for each heartbeat.
Issue 链接 / Issue Link: #
📋 变更类型 / Type of Change
📝 变更目的 / Purpose of the Change
避免偶然出现的 fitable 调用失败 / Avoid occasional fitable call failures
📋 主要变更 / Brief Changelog
🧪 验证变更 / Verifying this Change
测试步骤 / Test Steps
测试覆盖 / Test Coverage
📸 截图 / Screenshots
✅ 贡献者检查清单 / Contributor Checklist
请确保你的 Pull Request 符合以下要求 / Please ensure your Pull Request meets the following requirements:
基本要求 / Basic Requirements:
代码质量 / Code Quality:
测试要求 / Testing Requirements:
mvn -B clean package -Dmaven.test.skip=true,elsa README 中的编译检查 / Basic checks passmvn clean install/ Unit tests pass文档和兼容性 / Documentation and Compatibility:
📋 附加信息 / Additional Notes
审查者注意事项 / Reviewer Notes: