Skip to content

Commit

Permalink
fix panic caused by string is nil (#169)
Browse files Browse the repository at this point in the history
* fix panic caused by string is nil

* remove print tableParam
  • Loading branch information
GroundWu authored Jul 18, 2024
1 parent 044f1e3 commit b2deb92
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions client/obclient.go
Original file line number Diff line number Diff line change
Expand Up @@ -615,9 +615,10 @@ func (c *obClient) execute(
// 1. Get table route
tableParam, err := c.GetTableParam(ctx, tableName, rowKey)
if err != nil {
isOdpMode := c.odpTable != nil
log.Error("Runtime", ctx.Value(log.ObkvTraceIdName), "error occur in execute",
log.Int64("opType", int64(opType)), log.String("tableName", tableName), log.String("tableParam", tableParam.String()))
return nil, errors.WithMessagef(err, "get table param, tableName:%s, opType:%d", tableName, opType), needRetry
log.Int64("opType", int64(opType)), log.String("tableName", tableName), log.Bool("isOdpMode", isOdpMode))
return nil, errors.WithMessagef(err, "get table param, tableName:%s, opType:%d, isOdpMode:%v", tableName, opType, isOdpMode), needRetry
}

// 2. Construct request.
Expand Down

0 comments on commit b2deb92

Please sign in to comment.