yangwwei commented on pull request #213:
URL: 
https://github.com/apache/incubator-yunikorn-core/pull/213#issuecomment-703947249


   Add a default placement rule sounds like a good idea, that gives us a more 
consistent way of handling app submission. This will be a bigger change.
   
   But even we are using that approach, I guess it is still needed to fix the 
NPE in `createSchedulingQueue ()`, correct? I mean this could happen even in 
the case while calling by the placement rule logic. @wilfred-s  could you 
please take another look and let us know if this is needed?
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to