On Tue, Mar 22, 2022 at 7:24 AM Andres Freund <and...@anarazel.de> wrote: > > Hi, > > On 2022-02-24 12:26:08 +0530, Sadhuprasad Patro wrote: > > I have added a dummy test module for table AM and did the document > > change in the latest patch attached... > > The test module doesn't build on windows, unfortunately... Looks like you need > to add PGDLLIMPORT to a few variables: > [01:26:18.539] > c:\cirrus\src\test\modules\dummy_table_am\dummy_table_am.c(488): warning > C4700: uninitialized local variable 'rel' used > [c:\cirrus\dummy_table_am.vcxproj] > [01:26:18.539] dummy_table_am.obj : error LNK2001: unresolved external symbol > synchronize_seqscans [c:\cirrus\dummy_table_am.vcxproj] > [01:26:18.539] .\Debug\dummy_table_am\dummy_table_am.dll : fatal error > LNK1120: 1 unresolved externals [c:\cirrus\dummy_table_am.vcxproj] > [01:26:18.539] 1 Warning(s) > [01:26:18.539] 2 Error(s) > > https://cirrus-ci.com/task/5067519584108544?logs=build#L2085 > > Marked the CF entry as waiting-on-author.
HI, Thank you for the feedback Andres. I will take care of the same. As of now attached is a new patch on this to support the addition of new option parameters or drop the old parameters through ALTER TABLE command. Need some more testing on this, which is currently in progress. Providing the patch to get early feedback in case of any major comments... New Command: ALTER TABLE name SET ACCESS METHOD amname [ OPTIONS ( ADD | DROP option 'value' [, ... ] ) ]; Thanks & Regards SadhuPrasad http://www.EnterpriseDB.com
v4-0001-PATCH-V4-Per-table-storage-parameters-for-TableAM.patch
Description: Binary data
