From 74e4c93a1f333352914314b3a53172a0164a20e9 Mon Sep 17 00:00:00 2001
From: radhika <radhika@curoverse.com>
Date: Thu, 19 Nov 2015 15:23:25 -0500
Subject: [PATCH] 7490: remove loggerutil.LogErrorMessage that I added since it
 is not used

---
 services/datamanager/collection/collection.go | 2 +-
 services/datamanager/datamanager.go           | 1 -
 services/datamanager/loggerutil/loggerutil.go | 9 ---------
 3 files changed, 1 insertion(+), 11 deletions(-)

diff --git a/services/datamanager/collection/collection.go b/services/datamanager/collection/collection.go
index f0f94339de..7102ec3c05 100644
--- a/services/datamanager/collection/collection.go
+++ b/services/datamanager/collection/collection.go
@@ -96,7 +96,7 @@ func WriteHeapProfile() error {
 }
 
 // GetCollectionsAndSummarize gets collections from api and summarizes
-func GetCollectionsAndSummarize(arvLogger *logger.Logger, params GetCollectionsParams) (results ReadCollections) {
+func GetCollectionsAndSummarize(params GetCollectionsParams) (results ReadCollections) {
 	results, err := GetCollections(params)
 	if err != nil {
 		results.Err = err
diff --git a/services/datamanager/datamanager.go b/services/datamanager/datamanager.go
index 50213787aa..3c53b4ad2a 100644
--- a/services/datamanager/datamanager.go
+++ b/services/datamanager/datamanager.go
@@ -183,7 +183,6 @@ func BuildDataFetcher(arv arvadosclient.ArvadosClient) summary.DataFetcher {
 
 		go func() {
 			collectionChannel <- collection.GetCollectionsAndSummarize(
-				arvLogger,
 				collection.GetCollectionsParams{
 					Client:    arv,
 					Logger:    arvLogger,
diff --git a/services/datamanager/loggerutil/loggerutil.go b/services/datamanager/loggerutil/loggerutil.go
index 453f7e16bc..8c655cd5ff 100644
--- a/services/datamanager/loggerutil/loggerutil.go
+++ b/services/datamanager/loggerutil/loggerutil.go
@@ -50,12 +50,3 @@ func FatalWithMessage(arvLogger *logger.Logger, message string) {
 
 	log.Fatalf(message)
 }
-
-func LogErrorMessage(arvLogger *logger.Logger, message string) {
-	if arvLogger != nil {
-		arvLogger.Update(func(p map[string]interface{}, e map[string]interface{}) {
-			runInfo := logger.GetOrCreateMap(p, "run_info")
-			runInfo["ERROR"] = message
-		})
-	}
-}
-- 
2.30.2