[ARVADOS] updated: 74e4c93a1f333352914314b3a53172a0164a20e9

git at public.curoverse.com git at public.curoverse.com
Thu Nov 19 15:24:18 EST 2015


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

       via  74e4c93a1f333352914314b3a53172a0164a20e9 (commit)
      from  07f18157a5f8b72c20c0050c4617564effe91bd2 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 74e4c93a1f333352914314b3a53172a0164a20e9
Author: radhika <radhika at curoverse.com>
Date:   Thu Nov 19 15:23:25 2015 -0500

    7490: remove loggerutil.LogErrorMessage that I added since it is not used

diff --git a/services/datamanager/collection/collection.go b/services/datamanager/collection/collection.go
index f0f9433..7102ec3 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 5021378..3c53b4a 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 453f7e1..8c655cd 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
-		})
-	}
-}

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list