markusthoemmes commented on a change in pull request #2531: Share bookkeeping
data across controllers
URL:
https://github.com/apache/incubator-openwhisk/pull/2531#discussion_r138258903
##########
File path: common/scala/src/main/scala/whisk/core/WhiskConfig.scala
##########
@@ -129,21 +131,16 @@ object WhiskConfig {
* Reads a Map of key-value pairs from the environment (sys.env) -- store
them in the
* mutable properties object.
*/
- def readPropertiesFromFile(properties: scala.collection.mutable.Map[String,
String], file: File)(
- implicit logging: Logging) = {
+ def readPropertiesFromFile(properties: scala.collection.mutable.Map[String,
String], file: File) = {
if (file != null && file.exists) {
- logging.info(this, s"reading properties from file $file")
for (line <- Source.fromFile(file).getLines if line.trim != "") {
val parts = line.split('=')
if (parts.length >= 1) {
val p = parts(0).trim
val v = if (parts.length == 2) parts(1).trim else ""
if (properties.contains(p)) {
properties += p -> v
- logging.debug(this, s"properties file set value for $p")
}
- } else {
- logging.warn(this, s"ignoring properties $line")
Review comment:
Do we lose anything by removing this? Like debuggability of failed startups?
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
