zhaohai666 commented on PR #1039: URL: https://github.com/apache/rocketmq-dashboard/pull/1039#issuecomment-5211773907
> 感谢你的工作！但我们无法直接合并这项更改： > > **1. 对于一个没有实现的 SPI 来说，存在太多接近空的 VO。** > > 这个 PR 包含了 `<provider>` `CloudInstanceVO`、`CloudEndpointVO`` <providerType> `CloudVendorVO`` 和一个`CloudVendor`枚举，但唯一的提供程序是 `<provider>` `NoopCloudMetadataProvider`，因此`/api/cloud/vendors``// `/api/cloud/instances``总是返回空结果或不可用结果。这与我们之前关闭的#659`/api/cloud/endpoints`问题如出一辙：在实际功能尚未实现之前就发布了无效的 API 接口。此外，`<provider>`只是简单地复制了枚举字段（vendor/displayName/providerType/available），而没有添加任何实际值。[](https://github.com/apache/rocketmq-dashboard/pull/659)`CloudVendorVO``CloudVendor` > > **2. 该功能应至少配备一个真正的适配器。** > > 仅包含 SPI 的阶段不适合单独合并：没有具体的提供商，合约无法验证；而且一旦实际的云控制平面桥接完成，VO 通常需要重新设计。请重新提交 SPI，并附上至少一个可用的供应商适配器（或者等到适配器准备就绪后再提交），并根据该适配器的实际需求精简模型。 > > **3. 小问题**：删除已禁用的`instance/topic/CloudMetadataProvider`存根是可以的，可以保留。 > > 中文说明： > > 1. **VO 过多只是且无实现支撑**：PR 引入了`CloudInstanceVO`//四个模型，但唯一实现的是`CloudEndpointVO`，三个端点永远返回空/不可用——与#659关闭的“先铺死 API”模式相同。也把枚举字段原样搬过来重复，没有赋值。`CloudVendorVO``CloudVendor``NoopCloudMetadataProvider``/api/cloud/*`[](https://github.com/apache/rocketmq-dashboard/pull/659)`CloudVendorVO``CloudVendor` > 2. **至少带一个真实的适配器再合入**：纯SPI阶段的契约无法验证，VO一旦接真实的云控制面大概率要返工。请至少带一个可用的厂商适配器一起重提，并将模型修剪到该适配器需要实际的字段。 > 3. 拆除已损坏的`instance/topic/CloudMetadataProvider`桩没问题，可保留。 Fixed -- 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]
