fix: Include table name in logs on panic (#505)

disq · web-flow · commit a0b8a46c05b8 · 2022-12-16T08:05:29.000Z
diff --git a/plugins/source_scheduler_dfs.go b/plugins/source_scheduler_dfs.go
@@ -92,7 +92,7 @@ func (p *SourcePlugin) resolveTableDfs(ctx context.Context, table *schema.Table,
 					scope.SetTag("table", table.Name)
 					sentry.CurrentHub().CaptureMessage(stack)
 				})
-				p.logger.Error().Interface("error", err).Str("stack", stack).Msg("table resolver finished with panic")
+				logger.Error().Interface("error", err).Str("stack", stack).Msg("table resolver finished with panic")
 				atomic.AddUint64(&tableMetrics.Panics, 1)
 			}
 			close(res)

Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,7 @@ func (p SourcePlugin) resolveTableDfs(ctx context.Context, table schema.Table,`
`92`	`92`	`scope.SetTag("table", table.Name)`
`93`	`93`	`sentry.CurrentHub().CaptureMessage(stack)`
`94`	`94`	`})`
`95`		`- p.logger.Error().Interface("error", err).Str("stack", stack).Msg("table resolver finished with panic")`
	`95`	`+ logger.Error().Interface("error", err).Str("stack", stack).Msg("table resolver finished with panic")`
`96`	`96`	`atomic.AddUint64(&tableMetrics.Panics, 1)`
`97`	`97`	`}`
`98`	`98`	`close(res)`