TisonKun commented on a change in pull request #7809: [FLINK-11710][tests]
Refactor SimpleSlotProvider to TestingLogicalSlo…
URL: https://github.com/apache/flink/pull/7809#discussion_r259903358
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/slotpool/AllocatedSlot.java
##########
@@ -41,7 +41,7 @@
* an AllocatedSlot was allocated to the JobManager as soon as the TaskManager
registered at the
* JobManager. All slots had a default unknown resource profile.
*/
-class AllocatedSlot implements PhysicalSlot {
+public class AllocatedSlot implements PhysicalSlot {
Review comment:
I prefer rename `SimpleSlotContext` to `SettableSlotContext` and use it in
this pr, as a follow up, also drop `DummySlotContext`. What do you think?
Should it be a separated JIRA?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
