On 11/01/2018 07:43 AM, Amit Kapila wrote:

You can find the latest code at https://github.com/EnterpriseDB/zheap


Seems valgrind complains about a couple of places in the code - nothing major, might be noise, but probably worth a look.

regards

--
Tomas Vondra                  http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
==7569== Conditional jump or move depends on uninitialised value(s)
==7569==    at 0x5914C8: ZHeapDetermineModifiedColumns (zheapam.c:6666)
==7569==    by 0x587453: zheap_update (zheapam.c:2176)
==7569==    by 0x7577D3: ExecUpdate (nodeModifyTable.c:1426)
==7569==    by 0x759C0B: ExecModifyTable (nodeModifyTable.c:2629)
==7569==    by 0x729687: ExecProcNodeFirst (execProcnode.c:445)
==7569==    by 0x71D8E9: ExecProcNode (executor.h:241)
==7569==    by 0x720194: ExecutePlan (execMain.c:1711)
==7569==    by 0x71DF11: standard_ExecutorRun (execMain.c:367)
==7569==    by 0x71DD3F: ExecutorRun (execMain.c:310)
==7569==    by 0x9113CA: ProcessQuery (pquery.c:161)
==7569==    by 0x912CE8: PortalRunMulti (pquery.c:1286)
==7569==    by 0x9122C1: PortalRun (pquery.c:799)
==7569==    by 0x90C1C2: exec_simple_query (postgres.c:1215)
==7569==    by 0x9104D9: PostgresMain (postgres.c:4244)
==7569==    by 0x86A911: BackendRun (postmaster.c:4388)
==7569==    by 0x86A0DF: BackendStartup (postmaster.c:4079)
==7569==    by 0x8665F7: ServerLoop (postmaster.c:1711)
==7569==    by 0x865EA3: PostmasterMain (postmaster.c:1384)
==7569==    by 0x78E3AB: main (main.c:228)
==7569==  Uninitialised value was created by a stack allocation
==7569==    at 0x59043D: znocachegetattr (zheapam.c:6206)
==7569== 
{
   <insert_a_suppression_name_here>
   Memcheck:Cond
   fun:ZHeapDetermineModifiedColumns
   fun:zheap_update
   fun:ExecUpdate
   fun:ExecModifyTable
   fun:ExecProcNodeFirst
   fun:ExecProcNode
   fun:ExecutePlan
   fun:standard_ExecutorRun
   fun:ExecutorRun
   fun:ProcessQuery
   fun:PortalRunMulti
   fun:PortalRun
   fun:exec_simple_query
   fun:PostgresMain
   fun:BackendRun
   fun:BackendStartup
   fun:ServerLoop
   fun:PostmasterMain
   fun:main
}
==8811== Conditional jump or move depends on uninitialised value(s)
==8811==    at 0x5914C8: ZHeapDetermineModifiedColumns (zheapam.c:6666)
==8811==    by 0x587453: zheap_update (zheapam.c:2176)
==8811==    by 0x7577D3: ExecUpdate (nodeModifyTable.c:1426)
==8811==    by 0x759C0B: ExecModifyTable (nodeModifyTable.c:2629)
==8811==    by 0x729687: ExecProcNodeFirst (execProcnode.c:445)
==8811==    by 0x71D8E9: ExecProcNode (executor.h:241)
==8811==    by 0x720194: ExecutePlan (execMain.c:1711)
==8811==    by 0x71DF11: standard_ExecutorRun (execMain.c:367)
==8811==    by 0x71DD3F: ExecutorRun (execMain.c:310)
==8811==    by 0x9113CA: ProcessQuery (pquery.c:161)
==8811==    by 0x912CE8: PortalRunMulti (pquery.c:1286)
==8811==    by 0x9122C1: PortalRun (pquery.c:799)
==8811==    by 0x90C1C2: exec_simple_query (postgres.c:1215)
==8811==    by 0x9104D9: PostgresMain (postgres.c:4244)
==8811==    by 0x86A911: BackendRun (postmaster.c:4388)
==8811==    by 0x86A0DF: BackendStartup (postmaster.c:4079)
==8811==    by 0x8665F7: ServerLoop (postmaster.c:1711)
==8811==    by 0x865EA3: PostmasterMain (postmaster.c:1384)
==8811==    by 0x78E3AB: main (main.c:228)
==8811==  Uninitialised value was created by a stack allocation
==8811==    at 0x59043D: znocachegetattr (zheapam.c:6206)
==8811== 
{
   <insert_a_suppression_name_here>
   Memcheck:Cond
   fun:ZHeapDetermineModifiedColumns
   fun:zheap_update
   fun:ExecUpdate
   fun:ExecModifyTable
   fun:ExecProcNodeFirst
   fun:ExecProcNode
   fun:ExecutePlan
   fun:standard_ExecutorRun
   fun:ExecutorRun
   fun:ProcessQuery
   fun:PortalRunMulti
   fun:PortalRun
   fun:exec_simple_query
   fun:PostgresMain
   fun:BackendRun
   fun:BackendStartup
   fun:ServerLoop
   fun:PostmasterMain
   fun:main
}
==8811== Conditional jump or move depends on uninitialised value(s)
==8811==    at 0x574A76: InsertUndoRecord (undorecord.c:135)
==8811==    by 0x56F405: InsertPreparedUndo (undoinsert.c:840)
==8811==    by 0x5848E0: zheap_insert (zheapam.c:896)
==8811==    by 0x755F21: ExecInsert (nodeModifyTable.c:692)
==8811==    by 0x759BB2: ExecModifyTable (nodeModifyTable.c:2622)
==8811==    by 0x729687: ExecProcNodeFirst (execProcnode.c:445)
==8811==    by 0x71D8E9: ExecProcNode (executor.h:241)
==8811==    by 0x720194: ExecutePlan (execMain.c:1711)
==8811==    by 0x71DF11: standard_ExecutorRun (execMain.c:367)
==8811==    by 0x71DD3F: ExecutorRun (execMain.c:310)
==8811==    by 0x9113CA: ProcessQuery (pquery.c:161)
==8811==    by 0x912CE8: PortalRunMulti (pquery.c:1286)
==8811==    by 0x9122C1: PortalRun (pquery.c:799)
==8811==    by 0x90C1C2: exec_simple_query (postgres.c:1215)
==8811==    by 0x9104D9: PostgresMain (postgres.c:4244)
==8811==    by 0x86A911: BackendRun (postmaster.c:4388)
==8811==    by 0x86A0DF: BackendStartup (postmaster.c:4079)
==8811==    by 0x8665F7: ServerLoop (postmaster.c:1711)
==8811==    by 0x865EA3: PostmasterMain (postmaster.c:1384)
==8811==    by 0x78E3AB: main (main.c:228)
==8811==  Uninitialised value was created by a stack allocation
==8811==    at 0x5841D3: zheap_insert (zheapam.c:685)
==8811== 
{
   <insert_a_suppression_name_here>
   Memcheck:Cond
   fun:InsertUndoRecord
   fun:InsertPreparedUndo
   fun:zheap_insert
   fun:ExecInsert
   fun:ExecModifyTable
   fun:ExecProcNodeFirst
   fun:ExecProcNode
   fun:ExecutePlan
   fun:standard_ExecutorRun
   fun:ExecutorRun
   fun:ProcessQuery
   fun:PortalRunMulti
   fun:PortalRun
   fun:exec_simple_query
   fun:PostgresMain
   fun:BackendRun
   fun:BackendStartup
   fun:ServerLoop
   fun:PostmasterMain
   fun:main
}
==8811== Conditional jump or move depends on uninitialised value(s)
==8811==    at 0x574AA0: InsertUndoRecord (undorecord.c:136)
==8811==    by 0x56F405: InsertPreparedUndo (undoinsert.c:840)
==8811==    by 0x5848E0: zheap_insert (zheapam.c:896)
==8811==    by 0x755F21: ExecInsert (nodeModifyTable.c:692)
==8811==    by 0x759BB2: ExecModifyTable (nodeModifyTable.c:2622)
==8811==    by 0x729687: ExecProcNodeFirst (execProcnode.c:445)
==8811==    by 0x71D8E9: ExecProcNode (executor.h:241)
==8811==    by 0x720194: ExecutePlan (execMain.c:1711)
==8811==    by 0x71DF11: standard_ExecutorRun (execMain.c:367)
==8811==    by 0x71DD3F: ExecutorRun (execMain.c:310)
==8811==    by 0x9113CA: ProcessQuery (pquery.c:161)
==8811==    by 0x912CE8: PortalRunMulti (pquery.c:1286)
==8811==    by 0x9122C1: PortalRun (pquery.c:799)
==8811==    by 0x90C1C2: exec_simple_query (postgres.c:1215)
==8811==    by 0x9104D9: PostgresMain (postgres.c:4244)
==8811==    by 0x86A911: BackendRun (postmaster.c:4388)
==8811==    by 0x86A0DF: BackendStartup (postmaster.c:4079)
==8811==    by 0x8665F7: ServerLoop (postmaster.c:1711)
==8811==    by 0x865EA3: PostmasterMain (postmaster.c:1384)
==8811==    by 0x78E3AB: main (main.c:228)
==8811==  Uninitialised value was created by a stack allocation
==8811==    at 0x5841D3: zheap_insert (zheapam.c:685)
==8811== 
{
   <insert_a_suppression_name_here>
   Memcheck:Cond
   fun:InsertUndoRecord
   fun:InsertPreparedUndo
   fun:zheap_insert
   fun:ExecInsert
   fun:ExecModifyTable
   fun:ExecProcNodeFirst
   fun:ExecProcNode
   fun:ExecutePlan
   fun:standard_ExecutorRun
   fun:ExecutorRun
   fun:ProcessQuery
   fun:PortalRunMulti
   fun:PortalRun
   fun:exec_simple_query
   fun:PostgresMain
   fun:BackendRun
   fun:BackendStartup
   fun:ServerLoop
   fun:PostmasterMain
   fun:main
}

Reply via email to