wilfred-s commented on a change in pull request #251:
URL:
https://github.com/apache/incubator-yunikorn-core/pull/251#discussion_r583248064
##########
File path: pkg/scheduler/partition.go
##########
@@ -380,9 +379,6 @@ func (pc *PartitionContext) removeApplication(appID string)
[]*objects.Allocatio
}
}
}
- log.Logger().Debug("application removed from the scheduler",
- zap.String("queue", queueName),
- zap.String("applicationID", appID))
Review comment:
We have more logs now, as part of the change we log this in the context
at an INFO level to make sure we log all possible failure cases for adds and
the removed entry has been
[moved](https://github.com/apache/incubator-yunikorn-core/pull/251/files#diff-a6bde2f6900fb05707fa68db3365f08a79642f39ea629ad5b489d05d3cacc7b3R497-R499)
add and remove log the same way now
----------------------------------------------------------------
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]