Removed useless field from WorkerTask.

The original plan was to implement specific functionality in individual
functions, however now it seems that a single function rewrite_table() can
handel all the use cases. Thus we don't need anything like task kind.

Author: Antonin Houska

pg_rewrite.c

@@ -574,7 +574,6 @@ rewrite_table(PG_FUNCTION_ARGS)
 
 	task = get_task(&task_idx, rv_src->schemaname, rv_src->relname);
 	Assert(task_idx < MAX_TASKS);
-	task->kind = WORKER_TASK_PARTITION;
 
 	/* Fill-in the partitioning specific fields. */
 	if (rv_dst->schemaname)
@@ -673,7 +672,6 @@ rewrite_worker_main(Datum main_arg)
 	StartTransactionCommand();
 	PG_TRY();
 	{
-		Assert(MyWorkerTask->kind == WORKER_TASK_PARTITION);
 		rewrite_table_impl(relschema, relname, relname_new, relschema_dst,
 						   relname_dst);
 		CommitTransactionCommand();

pg_rewrite.h

@@ -200,12 +200,6 @@ typedef struct TaskProgress
 	int64		del;
 } TaskProgress;
 
-/* TODO Remove */
-typedef enum WorkerTaskKind
-{
-	WORKER_TASK_PARTITION
-} WorkerTaskKind;
-
 /*
  * The new implementation, which delegates the execution to a background
  * worker (as opposed to the PG executor).
@@ -215,8 +209,6 @@ typedef enum WorkerTaskKind
  */
 typedef struct WorkerTask
 {
-	WorkerTaskKind	kind;
-
 	/* Connection info. */
 	Oid		dbid;
 	Oid		roleid;