zhuzhurk commented on PR #27333: URL: https://github.com/apache/flink/pull/27333#issuecomment-3753138491
Agreed that `LoadingWeight` is a bit too abstract. From my understanding, the current `LoadingWeight` represents the load of task execution. At the moment, it is the number of tasks that will run in the given slot. Please correct me if I’m mistaken, @RocMarshal . To me, `SchedulingLoad` sounds more like it refers to the overhead of scheduling tasks, rather than the actual execution load. So perhaps `TaskExecutionLoad` would be a clearer and more precise name. More importantly, I think we should add more detailed comments to this class, so that one does not need to go through all the related code to find what it actually means. What do you think? @ferenc-csaky -- 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]
