From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Tue, 29 Nov 2016 16:07:02 +0000 (-0500)
Subject: Fix incorrect variable type in set_rel_consider_parallel().
X-Git-Tag: REL_10_BETA1~1357
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=d6c8b34e956864d52780f0db9f8cfe3b2f4411b0;p=postgresql

Fix incorrect variable type in set_rel_consider_parallel().

func_parallel() returns char not Oid.  Harmless, but still wrong.

Amit Langote
---

diff --git a/src/backend/optimizer/path/allpaths.c b/src/backend/optimizer/path/allpaths.c
index e42ef98d8d..9753a263fe 100644
--- a/src/backend/optimizer/path/allpaths.c
+++ b/src/backend/optimizer/path/allpaths.c
@@ -537,7 +537,7 @@ set_rel_consider_parallel(PlannerInfo *root, RelOptInfo *rel,
 			 */
 			if (rte->tablesample != NULL)
 			{
-				Oid			proparallel = func_parallel(rte->tablesample->tsmhandler);
+				char		proparallel = func_parallel(rte->tablesample->tsmhandler);
 
 				if (proparallel != PROPARALLEL_SAFE)
 					return;