On 2019/05/20 13:42, Rushabh Lathia wrote: > On Fri, May 17, 2019 at 10:30 AM Amit Langote <langote_amit...@lab.ntt.co.jp> >> Why not change it like this instead: >> >> @@ -681,7 +681,8 @@ DefineRelation(CreateStmt *stmt, char relkind, Oid >> ownerId, >> Assert(list_length(inheritOids) == 1); >> tablespaceId = get_rel_tablespace(linitial_oid(inheritOids)); >> } >> - else >> + >> + if (!OidIsValid(tablespaceId)) >> tablespaceId = >> GetDefaultTablespace(stmt->relation->relpersistence, >> partitioned); > > Yes, sure we can do that. Here is the patch for the same.
Thanks Rushabh. Regards, Amit