Re: [GSoC] working status - Mailing list pgsql-hackers

From Aleksander Alekseeev
Subject Re: [GSoC] working status
Date
Msg-id 20180709151604.55677bf4@e733.localdomain
Whole thread Raw
In response to Re: [GSoC] working status  (Aleksander Alekseeev <a.alekseev@postgrespro.ru>)
Responses Re: [GSoC] working status  (Aleksander Alekseeev <a.alekseev@postgrespro.ru>)
List pgsql-hackers
Hello Charles,

> ```
> pg_thrift.c:1313:26: error: too few arguments to function
> ‘array_create_iterator’ ArrayIterator iter =
> array_create_iterator(parray, 0); ^~~~~~~~~~~~~~~~~~~~~
> In file included from pg_thrift.c:5:
> .../postgresql-install/include/server/utils/array.h:418:22:
> note: declared here extern ArrayIterator
> array_create_iterator(ArrayType *arr, int slice_ndim, ArrayMetaState
> *mstate); ^~~~~~~~~~~~~~~~~~~~~
> ```

I should probably explain this in a little more detail.

This was an attempt to build pg_thrift with PostgreSQL 10 (compiled from
REL_10_STABLE branch). Same issue with upcoming 11 version. I believe
you are testing the extension on some 9.x branch and don't see these
errors. Corresponding change in the API was done quite a long time ago,
in 2015 (see commit 13dbc7a8).

To solve this issue you can either use #ifdef's and maintain two
versions of the code for different versions of PostgreSQL or just
support only 10+. Both options are fine, at least by me personally.

--
Best regards,
Aleksander Alekseev


pgsql-hackers by date:

Previous
From: Aleksander Alekseeev
Date:
Subject: Re: [GSoC] working status
Next
From: Aleksander Alekseeev
Date:
Subject: Re: [GSoC] working status