Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1393#discussion_r212225863
--- Diff: src/include/nodes/memnodes.h ---
@@ -66,9 +66,7 @@ typedef struct MemoryContextMethods
void (*stats) (MemoryContext context, uint64 *nBlocks,
uint64 *nChunks, uint64 *currentAvailable, uint64 *allAllocated, uint64
*allFreed, uint64 *maxHeld);
void (*release_accounting)(MemoryContext context);
void (*update_generation)(MemoryContext context);
-#ifdef MEMORY_CONTEXT_CHECKING
--- End diff --
why remove these memory context checking ?
---