nkhorman / json_fdw

PostgreSQL extension which implements a Foreign Data Wrapper (FDW) for JSON files.
109 stars 12 forks source link

Not compatible with PostgreSQL 9.5 Alpha 2 #5

Open jmealo opened 9 years ago

jmealo commented 9 years ago

When trying to install using PGXN compilation fails with the following error:

Installing postgresql extensions using PGXN: json_fdw... installing
INFO: best version: json_fdw 1.0.0
INFO: saving /tmp/tmp81LHJo/json_fdw-1.0.0.zip
INFO: unpacking: /tmp/tmp81LHJo/json_fdw-1.0.0.zip
INFO: building extension
gcc -Wall -Wmissing-prototypes -Wpointer-arith -Wdeclaration-after-statement -Wendif-labels -Wmissing-format-attribute -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -g -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security -I/usr/include/mit-krb5 -fPIC -pie -fno-omit-frame-pointer -fpic -I. -I./ -I/usr/include/postgresql/9.5/server -I/usr/include/postgresql/internal -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -I/usr/include/libxml2  -I/usr/include/tcl8.6  -c -o json_fdw.o json_fdw.c
json_fdw.c: In function ‘JsonGetForeignPlan’:
json_fdw.c:313:12: error: too few arguments to function ‘make_foreignscan’
            foreignPrivateList);
            ^
In file included from json_fdw.c:35:0:
/usr/include/postgresql/9.5/server/optimizer/planmain.h:46:21: note: declared here
 extern ForeignScan *make_foreignscan(List *qptlist, List *qpqual,
                     ^
make: *** [json_fdw.o] Error 1

Thanks & Kudos, Jeff

ozgune commented 9 years ago

@jmealo Yes, your observation is correct. With the new JSONB data type in PostgreSQL 9.4, json_fdw is certainly less applicable than it used to be.

We don't know at this stage whether we'll port it to PostgreSQL 9.5 once the official release comes out.

Do you have a specific use-case for json_fdw?

Best, Ozgun

metdos commented 8 years ago

Hey @jmealo,

This project is retired and currently not recommended as a base for new consumers. If you're looking to query semi-structured data, please check the new JSON and JSONB data types built into PostgreSQL.