featzhang commented on PR #27771: URL: https://github.com/apache/flink/pull/27771#issuecomment-4062587654
Closing this PR due to fundamental architectural issues found during CI analysis. **Issues Identified:** 1. Access to private members (MetricStore.JobMetricStore, TaskMetricStore.subtasks) 2. Non-existent API (JobMetricStoreSnapshot.get()) 3. Incorrect interface usage (MessageHeaders instead of RuntimeMessageHeaders) 4. Wrong class references (HttpMethod instead of HttpMethodWrapper) 5. Incorrect Handler inheritance structure **Root Cause:** The implementation does not follow the standard Flink REST API architecture pattern. A complete redesign is required. **Next Steps:** - Research existing metrics handler implementations (e.g., JobManagerOperatorMetricsHandler) - Redesign the architecture following Flink standards - Reimplement with proper interfaces and access patterns -- 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]
