modify logger for enable prefix (#97)

* modify logger for enable prefix

* fix bug

tars/remotelogger.go

@@ -13,6 +13,7 @@ type RemoteTimeWriter struct {
 	logPtr        *logf.Log
 	reportSuccPtr *PropertyReport
 	reportFailPtr *PropertyReport
+	hasPrefix     bool
 }
 
 //NewRemoteTimeWriter new and init RemoteTimeWriter
@@ -135,7 +136,11 @@ func (rw *RemoteTimeWriter) InitFormat(s string) {
 
 //NeedPrefix return if need prefix for the logger.
 func (rw *RemoteTimeWriter) NeedPrefix() bool {
-	return false
+	return rw.hasPrefix
+}
+
+func (rw *RemoteTimeWriter) SetPrefix(enable bool) {
+	rw.hasPrefix = enable
 }
 
 //Write Writes the logs to the buffer.

tars/servant.go

@@ -71,7 +71,7 @@ func (s *ServantProxy) Tars_invoke(ctx context.Context, ctype byte,
 		err = s.obj.Invoke(ctx, msg, time.Duration(s.timeout)*time.Millisecond)
 	}
 	if err != nil {
-		TLOG.Error("Invoke error:", s.name, sFuncName, err.Error())
+		TLOG.Errorf("Invoke Obj:%s,fun:%s,error:%s", s.name, sFuncName, err.Error())
 		if msg.Resp == nil {
 			ReportStat(msg, 0, 0, 1)
 		} else if msg.Status == basef.TARSINVOKETIMEOUT {

tars/statf.go

@@ -163,7 +163,7 @@ func ReportStatFromClient(msg *Message, succ int32, timeout int32, exec int32) {
 	head.InterfaceName = msg.Req.SFuncName
 	sNames := strings.Split(msg.Req.SServantName, ".")
 	if len(sNames) < 2 {
-		TLOG.Debug("report err:servant name (%s) format error", msg.Req.SServantName)
+		TLOG.Debugf("report err:servant name (%s) format error", msg.Req.SServantName)
 		return
 	}
 	head.SlaveName = fmt.Sprintf("%s.%s", sNames[0], sNames[1])

tars/util/rogger/logger.go

@@ -234,9 +234,10 @@ func (l *Logger) writef(level LogLevel, format string, v []interface{}) {
 			}
 			fmt.Fprintf(buf, "%s:%d|", file, line)
 		}
+
+		buf.WriteString(level.String())
+		buf.WriteByte('|')
 	}
-	buf.WriteString(level.String())
-	buf.WriteByte('|')
 
 	if format == "" {
 		fmt.Fprint(buf, v...)

tars/util/rogger/logwriter.go

@@ -37,13 +37,14 @@ type RollFileWriter struct {
 
 //DateWriter rotate logs by date.
 type DateWriter struct {
-	logpath  string
-	name     string
-	dateType DateType
-	num      int
-	currDate string
-	currFile *os.File
-	openTime int64
+	logpath   string
+	name      string
+	dateType  DateType
+	num       int
+	currDate  string
+	currFile  *os.File
+	openTime  int64
+	hasPrefix bool
 }
 
 //HourWriter for rotate logs by hour
@@ -147,16 +148,21 @@ func (w *DateWriter) Write(v []byte) {
 
 //NeedPrefix shows whether needs prefix info for DateWriter or not.
 func (w *DateWriter) NeedPrefix() bool {
-	return true
+	return w.hasPrefix
+}
+
+func (w *DateWriter) SetPrefix(enable bool) {
+	w.hasPrefix = enable
 }
 
 //NewDateWriter returns a writer which keeps logs in hours or day format.
 func NewDateWriter(logpath, name string, dateType DateType, num int) *DateWriter {
 	w := &DateWriter{
-		logpath:  logpath,
-		name:     name,
-		num:      num,
-		dateType: dateType,
+		logpath:   logpath,
+		name:      name,
+		num:       num,
+		dateType:  dateType,
+		hasPrefix: true,
 	}
 	w.currDate = w.getCurrDate()
 	return w