mirror of https://github.com/citusdata/citus.git
Addresses review items
parent
3244c3967f
commit
0db68de575
|
@ -95,13 +95,13 @@ int UtilityHookLevel = 0;
|
||||||
|
|
||||||
|
|
||||||
/* Local functions forward declarations for helper functions */
|
/* Local functions forward declarations for helper functions */
|
||||||
static void ProcessUtilityInternal(PlannedStmt *pstmt,
|
static void citus_ProcessUtilityInternal(PlannedStmt *pstmt,
|
||||||
const char *queryString,
|
const char *queryString,
|
||||||
ProcessUtilityContext context,
|
ProcessUtilityContext context,
|
||||||
ParamListInfo params,
|
ParamListInfo params,
|
||||||
struct QueryEnvironment *queryEnv,
|
struct QueryEnvironment *queryEnv,
|
||||||
DestReceiver *dest,
|
DestReceiver *dest,
|
||||||
QueryCompletion *completionTag);
|
QueryCompletion *completionTag);
|
||||||
static void set_indexsafe_procflags(void);
|
static void set_indexsafe_procflags(void);
|
||||||
static char * CurrentSearchPath(void);
|
static char * CurrentSearchPath(void);
|
||||||
static void IncrementUtilityHookCountersIfNecessary(Node *parsetree);
|
static void IncrementUtilityHookCountersIfNecessary(Node *parsetree);
|
||||||
|
@ -329,8 +329,8 @@ citus_ProcessUtility(PlannedStmt *pstmt,
|
||||||
|
|
||||||
PG_TRY();
|
PG_TRY();
|
||||||
{
|
{
|
||||||
ProcessUtilityInternal(pstmt, queryString, context, params, queryEnv, dest,
|
citus_ProcessUtilityInternal(pstmt, queryString, context, params, queryEnv, dest,
|
||||||
completionTag);
|
completionTag);
|
||||||
|
|
||||||
if (UtilityHookLevel == 1)
|
if (UtilityHookLevel == 1)
|
||||||
{
|
{
|
||||||
|
@ -404,7 +404,7 @@ citus_ProcessUtility(PlannedStmt *pstmt,
|
||||||
|
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* ProcessUtilityInternal is a helper function for citus_ProcessUtility where majority
|
* citus_ProcessUtilityInternal is a helper function for citus_ProcessUtility where majority
|
||||||
* of the Citus specific utility statements are handled here. The distinction between
|
* of the Citus specific utility statements are handled here. The distinction between
|
||||||
* both functions is that Citus_ProcessUtility does not handle CALL and DO statements.
|
* both functions is that Citus_ProcessUtility does not handle CALL and DO statements.
|
||||||
* The reason for the distinction is implemented to be able to find the "top-level" DDL
|
* The reason for the distinction is implemented to be able to find the "top-level" DDL
|
||||||
|
@ -412,13 +412,13 @@ citus_ProcessUtility(PlannedStmt *pstmt,
|
||||||
* this goal.
|
* this goal.
|
||||||
*/
|
*/
|
||||||
static void
|
static void
|
||||||
ProcessUtilityInternal(PlannedStmt *pstmt,
|
citus_ProcessUtilityInternal(PlannedStmt *pstmt,
|
||||||
const char *queryString,
|
const char *queryString,
|
||||||
ProcessUtilityContext context,
|
ProcessUtilityContext context,
|
||||||
ParamListInfo params,
|
ParamListInfo params,
|
||||||
struct QueryEnvironment *queryEnv,
|
struct QueryEnvironment *queryEnv,
|
||||||
DestReceiver *dest,
|
DestReceiver *dest,
|
||||||
QueryCompletion *completionTag)
|
QueryCompletion *completionTag)
|
||||||
{
|
{
|
||||||
Node *parsetree = pstmt->utilityStmt;
|
Node *parsetree = pstmt->utilityStmt;
|
||||||
List *ddlJobs = NIL;
|
List *ddlJobs = NIL;
|
||||||
|
|
Loading…
Reference in New Issue