fix: return false instead of panic for test mock IsScatterUpdateLimitPassthruEnabled

tanjinx · Claude · tanjinx · commit 7ae63fb3f8af · 2026-04-09T16:36:07.000-07:00
Co-Authored-By: Claude &lt;svc-devxp-claude@slack-corp.com&gt;
Signed-off-by: Tanjin Xu &lt;tanjin.xu@slack-corp.com&gt;
diff --git a/go/vt/vtgate/planbuilder/plancontext/planning_context_test.go b/go/vt/vtgate/planbuilder/plancontext/planning_context_test.go
@@ -337,8 +337,7 @@ func (v *vschema) IsShardRoutingEnabled() bool {
 }
 
 func (v *vschema) IsScatterUpdateLimitPassthruEnabled() bool {
-	// TODO implement me
-	panic("implement me")
+	return false
 }
 
 func (v *vschema) IsViewsEnabled() bool {

Original file line number	Diff line number	Diff line change
`@@ -337,8 +337,7 @@ func (v *vschema) IsShardRoutingEnabled() bool {`
`337`	`337`	`}`
`338`	`338`
`339`	`339`	`func (v *vschema) IsScatterUpdateLimitPassthruEnabled() bool {`
`340`		`- // TODO implement me`
`341`		`- panic("implement me")`
	`340`	`+ return false`
`342`	`341`	`}`
`343`	`342`
`344`	`343`	`func (v *vschema) IsViewsEnabled() bool {`