Xiao-zhen-Liu commented on code in PR #8772:
URL: https://github.com/apache/texera/pull/8772#discussion_r4199579855
##########
computing-unit-managing-service/src/main/scala/org/apache/texera/service/util/ComputingUnitManagingServiceException.scala:
##########
@@ -37,7 +37,8 @@ final case class InsufficientComputingResource(resourceType:
String)
// User has reached the per-user CU-quota (number of running units)
final case class InsufficientComputingUnitQuota(maxNumberOfComputingUnit: Int)
extends ComputingUnitManagingServiceException(
- s"You may only have $maxNumberOfComputingUnit computing-unit(s) running
at the same time"
+ s"You may only have $maxNumberOfComputingUnit computing-unit(s) running
at the same time. " +
Review Comment:
Nit: the first sentence writes "computing-unit(s)" with a hyphen and the new
one "computing units". Could the first one become "computing unit(s)", so both
spell it the same way? The "(s)" is worth keeping, since the limit can be set
to 1. The expected string in the spec changes with it.
--
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]
