From 80aac8a36d06241121270a19023f9743de2ed0f2 Mon Sep 17 00:00:00 2001
From: Robin Thomas <rthomas900@gmail.com>
Date: Fri, 12 Aug 2016 10:10:42 -0400
Subject: [PATCH] removed unused variable and mix of declaration and code.
 (Sorry, my own dev environment was somehow no issuing these warnings, but
 Travis build did.)

---
 src/backend/distributed/planner/multi_physical_planner.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/backend/distributed/planner/multi_physical_planner.c b/src/backend/distributed/planner/multi_physical_planner.c
index 457ced0ec..68d2e27dd 100644
--- a/src/backend/distributed/planner/multi_physical_planner.c
+++ b/src/backend/distributed/planner/multi_physical_planner.c
@@ -3951,7 +3951,6 @@ FragmentAlias(RangeTblEntry *rangeTableEntry, RangeTableFragment *fragment)
 	CitusRTEKind fragmentType = fragment->fragmentType;
 	if (fragmentType == CITUS_RTE_RELATION)
 	{
-		char *shardAliasName = NULL;
 		ShardInterval *shardInterval = (ShardInterval *) fragment->fragmentReference;
 		uint64 shardId = shardInterval->shardId;
 
@@ -3974,9 +3973,8 @@ FragmentAlias(RangeTblEntry *rangeTableEntry, RangeTableFragment *fragment)
 		/*
 		 * Set shard name in alias to <relation_name>_<shard_id>.
 		 */
-		char *shardName = pstrdup(relationName);
-		AppendShardIdToName(&shardName, shardId);
-		fragmentName = shardName;
+		fragmentName = pstrdup(relationName);
+		AppendShardIdToName(&fragmentName, shardId);
 	}
 	else if (fragmentType == CITUS_RTE_REMOTE_QUERY)
 	{