From: | Charles Cui <charles(dot)cui1984(at)gmail(dot)com> |
---|---|
To: | Aleksander Alekseev <a(dot)alekseev(at)postgrespro(dot)ru> |
Cc: | Aleksandr Parfenov <a(dot)parfenov(at)postgrespro(dot)ru>, Anastasia Lubennikova <a(dot)lubennikova(at)postgrespro(dot)ru>, Andrey Borodin <x4mmm(at)yandex-team(dot)ru>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [GSoC] current working status |
Date: | 2018-06-15 14:58:32 |
Message-ID: | CA+SXE9vYc+5kKSZ2baEwDGMr1xG=V0p-zo5Z7fpG9vkRYtpQyQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
got it, i will resolve all these warnings. One wired thing when i set up CI
is the docker image seems having a postgres version which conflicts with my
test version. So, CI is broken now ,although tests pass at my local box. I
will also figure this out.
On Fri, Jun 15, 2018, 1:34 AM Aleksander Alekseeev <
a(dot)alekseev(at)postgrespro(dot)ru> wrote:
> 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://afiskon.ru/s/7a/fe681b17f0_paste.txt I suggest to rename
> the procedure to `convert_int8_to_char`.
>
> Also GCC still generates a lot of warnings (see my previous message).
> I would advise to fix all these warnings, otherwise you can miss a
> really important warning. Ideally, the code should be compiled with
> -Wall flag.
>
> --
> Best regards,
> Aleksander Alekseev
>
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2018-06-15 15:03:01 | Re: WAL prefetch |
Previous Message | Tom Lane | 2018-06-15 14:53:13 | Re: row_to_json(), NULL values, and AS |