)]}'
{"go/objectserver/replicator.go":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"7d1138e99013fb5b6e50c810afef1f2861c787fd","unresolved":false,"context_lines":[{"line_number":286,"context_line":"\t\thashDir :\u003d filepath.Dir(objFile)"},{"line_number":287,"context_line":"\t\tr.LogError(\"[syncFile] %s failed audit and is being quarantined: %s\", hashDir, err.Error())"},{"line_number":288,"context_line":"\t\tQuarantineHash(hashDir)"},{"line_number":289,"context_line":"\t\treturn 0, 0, nil"},{"line_number":290,"context_line":"\t} else if err !\u003d nil {"},{"line_number":291,"context_line":"\t\treturn 0, 0, nil"},{"line_number":292,"context_line":"\t}"}],"source_content_type":"text/x-go","patch_set":1,"id":"7aa08908_d1be3fff","line":289,"updated":"2016-06-16 20:16:02.000000000","message":"Dumb question: return error here or not (currently not). Seems to me this is okay, because the only place that cares at all is the replicateHandoff() and that one should continue to work as long as the number of insyncs returned is zero, which it is here. Just wondering if this was considered. Maybe no need to log here if returning an error.","commit_id":"d3024d947be319223868a9c4c4aca8bd02d3a066"}]}