Hello Charles,
>The second review is coming. Here is my working status so far. 1.
> Complete the thrift compact protocol implementation using bytea
> interface. 2. Thrift type (binary protocol) is almost done, the only
> remaining part is struct encoding and decoding. With the thrift type,
> y
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: dec
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
Hello Charles,
> Here is my current working status.
> 1. Complete the thrift_binary_in and thrift_binary_out functions, so
> that users can express their thrift struct using json. These two
> functions support both simple data struct and complex data structure
> like struct and map. 2. added t
Hello Charles,
> Having tried David's method to install 10.4 and 11 on my mac and
> turns out worked for me. The compiling issue posted by Aleksander is
> because some json helpers changed function name and is not backward
> compatible with 9.4 and 10. Using #if macro resolves the problem,
> Here
Hello Charles,
>The first evaluation is coming. Here is my progress so far. During
> the first stage of work, I have implemented the thrift binary
> protocol as the format of postgresql plugin. Currently, the main
> interface is byte. Users who use this plugin need to provide thrift
> bytes to
Hello Charles,
>I saw the list of errors you posted. That's because I have some new
> function implemented and pushed without testing(in order forget my
> change in local machine).
> So you are saying it is best to keep each commit workable. Right?
Ideally, yes. You can always create a separa
Hello Charles,
> The repo currently can be build on my mac. You can check it out.
> Also I am working on the CI config to monitor each commit.
Tests pass on Arch Linux with PostgreSQL 11 and Ubuntu 16.04 with
PostgreSQL 9.6. However there is a name conflict in case of PostgreSQL
11: https://afisk
Hello Charles,
>Here is my current working status. Resolved all warnings found by
> Aleksander previously. Having two threads in parallel. One is the
> thrift binary type implementation, the other is thrift compact byte
> interface implementation. For these two threads, simple data type has
>