Commit 87e0f9d

committed

Merge branch '189-disable-query-locks-in-chat' into 'master'

fix: temporary disable query locks in chat messages See merge request postgres-ai/joe!181

2 parents 65c510c + 1f61331 commit 87e0f9dCopy full SHA for 87e0f9d

File tree

-6

lines changed

-6

lines changed

Lines changed: 9 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ const (`
`34`	`34`	`queryExplainAnalyze = "EXPLAIN (ANALYZE, COSTS, VERBOSE, BUFFERS, FORMAT JSON) "`
`35`	`35`
`36`	`36`	`// locksTitle shows locks for a single query analyzed with EXPLAIN.`
`37`		`- locksTitle = "Query heavy locks:\n"`
	`37`	`+ // locksTitle = "Query heavy locks:\n".`
`38`	`38`	`)`
`39`	`39`
`40`	`40`	`// Explain runs an explain query.`
`@@ -117,13 +117,16 @@ func Explain(ctx context.Context, msgSvc connection.Messenger, command *platform`
`117`	`117`
`118`	`118`	`queryLocks := tableString.String()`
`119`	`119`	command.QueryLocks = strings.Trim(queryLocks, "`")
`120`		`- msg.AppendText(locksTitle + queryLocks)`
`121`	`120`
`122`		`- if err = msgSvc.UpdateText(msg); err != nil {`
`123`		`- log.Err("Show the plan with execution:", err)`
	`121`	`+ // TODO: Commented out because LISTEN/NOTIFY payload is limited to 8000 characters`
	`122`	`+ // Waiting for fixing: https://gitlab.com/postgres-ai/platform/-/merge_requests/255`
	`123`	`+ // msg.AppendText(locksTitle + queryLocks)`
`124`	`124`
`125`		`- return err`
`126`		`- }`
	`125`	`+ // if err = msgSvc.UpdateText(msg); err != nil {`
	`126`	`+ // log.Err("Show the plan with execution:", err)`
	`127`	`+ //`
	`128`	`+ // return err`
	`129`	`+ // }`
`127`	`130`
`128`	`131`	`if _, err := msgSvc.AddArtifact("plan-json", explainAnalyze, msg.ChannelID, msg.MessageID); err != nil {`
`129`	`132`	`log.Err("File upload failed:", err)`

Comments

(0)