On Thu, Mar 12, 2015 at 8:33 PM, Thom Brown <t...@linux.com> wrote: > > On 12 March 2015 at 14:46, Amit Kapila <amit.kapil...@gmail.com> wrote: > > One additional change (we need to SetLatch() in > > HandleParallelMessageInterrupt) > > is done to handle the hang issue reported on parallel-mode thread. > > Without this change it is difficult to verify the patch (will remove this > > change > > once new version of parallel-mode patch containing this change will be > > posted). > > Applied parallel-mode-v7.patch and parallel_seqscan_v10.patch, but > getting this error when building: > > gcc -Wall -Wmissing-prototypes -Wpointer-arith > -Wdeclaration-after-statement -Wendif-labels > -Wmissing-format-attribute -Wformat-security -fno-strict-aliasing > -fwrapv -fexcess-precision=standard -O2 -I../../../../src/include > -D_GNU_SOURCE -c -o brin.o brin.c -MMD -MP -MF .deps/brin.Po > In file included from ../../../../src/include/nodes/execnodes.h:18:0, > from ../../../../src/include/access/brin.h:14, > from brin.c:18: > ../../../../src/include/access/heapam.h:119:34: error: unknown type > name ‘ParallelHeapScanDesc’ > extern void heap_parallel_rescan(ParallelHeapScanDesc pscan, > HeapScanDesc scan); > ^ > > Am I missing another patch here?
Yes, the below parallel-heap-scan patch. http://www.postgresql.org/message-id/ca+tgmoyjetgeaxuszrona7bdtwzptqexpjntv1gkcavmgsd...@mail.gmail.com Please note that parallel_setup_cost and parallel_startup_cost are still set to zero by default, so you need to set it to higher values if you don't want the parallel plans once parallel_seqscan_degree is set. I have yet to comeup with default values for them, needs some tests. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com