[GitHub] rabbah commented on a change in pull request #2730: Reduce memory consumption for invocations of cached actions

[GitBox <gi...@apache.org>]

rabbah commented on a change in pull request #2730: Reduce memory consumption for invocations of cached actions
URL: https://github.com/apache/incubator-openwhisk/pull/2730#discussion_r146131628
 
 

 ##########
 File path: common/scala/src/main/scala/whisk/core/entity/Exec.scala
 ##########
 @@ -45,7 +45,7 @@ import whisk.core.entity.size.SizeString
  *   main  : name of the entry point function, when using a non-default value (for Java, the name of the main class)" }
  */
 sealed abstract class Exec extends ByteSizeable {
-  override def toString = Exec.serdes.write(this).compactPrint
+  override def toString: String = Exec.serdes.write(this).compactPrint
 
 Review comment:
   where do we use toString and should we replace with asString?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services