sql: add a TODO for sql-queries to investigate a test

After changing the sessiondata to stop using zero-value defaults, a test
started failing. It works when using a zero sessiondata, which deserves
more investigation.

Release note: None

pkg/sql/distsql_running_test.go

@@ -34,6 +34,8 @@ import (
 	"github.com/cockroachdb/cockroach/pkg/sql/parser"
 	"github.com/cockroachdb/cockroach/pkg/sql/rowenc"
 	"github.com/cockroachdb/cockroach/pkg/sql/sem/tree"
+	"github.com/cockroachdb/cockroach/pkg/sql/sessiondata"
+	"github.com/cockroachdb/cockroach/pkg/sql/sessiondatapb"
 	"github.com/cockroachdb/cockroach/pkg/testutils"
 	"github.com/cockroachdb/cockroach/pkg/testutils/pgtest"
 	"github.com/cockroachdb/cockroach/pkg/testutils/serverutils"
@@ -252,7 +254,14 @@ func TestDistSQLRunningParallelFKChecksAfterAbort(t *testing.T) {
 
 	createPlannerAndRunQuery := func(ctx context.Context, txn *kv.Txn, query string) error {
 		execCfg := s.ExecutorConfig().(ExecutorConfig)
-		sd := NewInternalSessionData(ctx, execCfg.Settings, "test")
+		// TODO(sql-queries): This sessiondata contains zero-values for most fields,
+		// meaning DistSQLMode is DistSQLOff. Is this correct?
+		sd := &sessiondata.SessionData{
+			SessionData:   sessiondatapb.SessionData{},
+			SearchPath:    sessiondata.DefaultSearchPathForUser(username.RootUserName()),
+			SequenceState: sessiondata.NewSequenceState(),
+			Location:      time.UTC,
+		}
 		// Plan the statement.
 		internalPlanner, cleanup := NewInternalPlanner(
 			"test",