tillrohrmann commented on a change in pull request #7809: [FLINK-11710][tests]
Refactor SimpleSlotProvider to TestingLogicalSlo…
URL: https://github.com/apache/flink/pull/7809#discussion_r259879291
##########
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:
For what do you need a subclass of `SlotContext`? There are already some
`SlotContext` test implementations. Maybe you could reuse one of these?
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
