{"comment":{"url":"https://api.github.com/repos/armenatzoglou/gpdb/comments/14464864","html_url":"https://github.com/armenatzoglou/gpdb/commit/512dd4dd5193b344846540427e54d41a8adc1cf4#commitcomment-14464864","id":14464864,"user":{"login":"hlinnaka","id":191602,"avatar_url":"https://avatars.githubusercontent.com/u/191602?v=3","gravatar_id":"","url":"https://api.github.com/users/hlinnaka","html_url":"https://github.com/hlinnaka","followers_url":"https://api.github.com/users/hlinnaka/followers","following_url":"https://api.github.com/users/hlinnaka/following{/other_user}","gists_url":"https://api.github.com/users/hlinnaka/gists{/gist_id}","starred_url":"https://api.github.com/users/hlinnaka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hlinnaka/subscriptions","organizations_url":"https://api.github.com/users/hlinnaka/orgs","repos_url":"https://api.github.com/users/hlinnaka/repos","events_url":"https://api.github.com/users/hlinnaka/events{/privacy}","received_events_url":"https://api.github.com/users/hlinnaka/received_events","type":"User","site_admin":false},"position":7,"line":878,"path":"src/backend/commands/sequence.c","commit_id":"512dd4dd5193b344846540427e54d41a8adc1cf4","created_at":"2015-11-18T08:08:32Z","updated_at":"2015-11-18T08:08:32Z","body":"A pstrdup() would be simpler than StringInfo here.\r\n\r\nActually, there is no need to close the relation before calling ereport() at all. Transaction abort will automatically release any locks. So you could just remove the relation_close() here, and pass RelationGetRelationName(seqrel) directly to ereport."}}