Re: Oddity in tuple routing for foreign partitions

[Etsuro Fujita]

(2018/05/02 15:45), Etsuro Fujita wrote:
(2018/05/02 10:10), Amit Langote wrote:
On 2018/05/02 6:09, Andres Freund wrote:
On 2018-05-01 13:41:32 -0400, Robert Haas wrote:
Committed.

Here is a small patch to remove a no-longer-needed cast in 
postgresBeginForeignInsert().

Best regards,
Etsuro Fujita
*** a/contrib/postgres_fdw/postgres_fdw.c
--- b/contrib/postgres_fdw/postgres_fdw.c
***************
*** 2007,2013 **** postgresBeginForeignInsert(ModifyTableState *mtstate,
  	/* Check if we add the ON CONFLICT clause to the remote query. */
  	if (plan)
  	{
! 		OnConflictAction onConflictAction = ((ModifyTable *) plan)->onConflictAction;
  
  		/* We only support DO NOTHING without an inference specification. */
  		if (onConflictAction == ONCONFLICT_NOTHING)
--- 2007,2013 ----
  	/* Check if we add the ON CONFLICT clause to the remote query. */
  	if (plan)
  	{
! 		OnConflictAction onConflictAction = plan->onConflictAction;
  
  		/* We only support DO NOTHING without an inference specification. */
  		if (onConflictAction == ONCONFLICT_NOTHING)