On Wed, Mar 22, 2017 at 7:57 PM, Ideriha, Takeshi <
ideriha.take...@jp.fujitsu.com> wrote:
> Hi, thank you very much for reviewing.
> Attached is v6 patch.
>
> >There was a minor conflict in applying 004_declareXX patch.
>
> I rebased 004_declareStmt_test_v6.patch.
>
> >Some comments in 001_declar
On Wed, Mar 22, 2017 at 12:51 PM, Haribabu Kommi
wrote:
>
>
> On Tue, Mar 7, 2017 at 4:09 PM, Ideriha, Takeshi <
> ideriha.take...@jp.fujitsu.com> wrote:
>
>>
>> Attached 004_declareStmt_test_v5.patch is a rebased one.
>> The rest of patches are same as older version.
>>
>
> Thanks for the update
On Tue, Mar 7, 2017 at 4:09 PM, Ideriha, Takeshi <
ideriha.take...@jp.fujitsu.com> wrote:
>
> Attached 004_declareStmt_test_v5.patch is a rebased one.
> The rest of patches are same as older version.
>
Thanks for the update patch. I started reviewing the patches.
There was a minor conflict in ap
Hi
I tried applying your patches. But it failed...
The error messages are as below.
$ git apply ../004_declareStmt_test_v4.patch
error: patch failed: src/interfaces/ecpg/test/expected/pgtypeslib-nan_test.c:82
error: src/interfaces/ecpg/test/expected/pgtypeslib-nan_test.c: patch does not
apply
R
On Wed, Jan 25, 2017 at 2:58 PM, Ideriha, Takeshi
wrote:
> Hi,
>
> I'm sorry but I think I don't have time to work on this patch in this CF.
> I've gotten busy for another work.
>
> So I moved this patch to next CF with "waiting on author" status.
Thanks for doing so, that's a time-saver for me!
gt; Sent: Friday, January 20, 2017 6:12 PM
> To: 'Michael Paquier'
> Cc: Michael Meskes ; pgsql-hackers@postgresql.org
> Subject: RE: [WIP] RE: [HACKERS] DECLARE STATEMENT setting up a connection in
> ECPG
>
> > Idehira-san, this is a very intrusive patch... It really
> Idehira-san, this is a very intrusive patch... It really needs a specific
> reviewer to begin with, but really it would be nice if you could review
> someone
> else's patch, with a difficulty rather equivalent to what we have here.
Michael, thank you for taking a look at my patch and giving me
On Fri, Jan 6, 2017 at 6:10 PM, Ideriha, Takeshi
wrote:
> Hi
> Thank you for looking over my patch.
>
>> Thank you. Would it be possible for you to re-create the patch without the
>> white-space changes?
>
> I'm sorry for adding redundant white-spaces.
> Attached is a correct version.
>
>> I notic
Hi Ideriha-san,
> I created a patch.
Thank you. Would it be possible for you to re-create the patch without
the white-space changes?
> I marked [WIP] to the title because some documentation is lacked and
> format needs some fixing.
I noticed that the docs say the statement is a PostgreSQL addo
Takeshi
> Cc: pgsql-hackers@postgresql.org
> Subject: Re: [HACKERS] DECLARE STATEMENT setting up a connection in ECPG
>
> > I wanted to say that in order to use the connection pointed by the
> > DECLARE STATEMENT some functions like ECPGdo() would be modified or
> >
> I wanted to say that in order to use the connection pointedÂ
> by the DECLARE STATEMENT some functions like ECPGdo() would be
> modified or
> new function would be added under the directory ecpglib/.
>
> This modification or new function will be used to get the connection
> by statement_name.
A
Thank you for your comment.
Friday, November 18, 2016 4:45 AM Michael Meskes wrote :
> > - Translate the DECLARE STATEMENT into a new function with parameters.
> >These parameters carry the information like connection_name and
> statement_name.
> > - The function is a new function defined in
> [System Design Plan]
> To support above functionality, ecpg precompiler should support:
> - To understand the DECLARE STATEMENT syntax
Already does, albeit as a noop.
> - Translate the DECLARE STATEMENT into a new function with parameters.
>These parameters carry the information like co
Hi,
Making developing ECPG application more efficiently and improve portability,
I'd like to suggest DECLARE STATEMENT support in ECPG.
This DECLARE STATEMENT is one of the statement that lets users declare an
identifier pointing a connection.
This identifier will be used in other embedded dynami
14 matches
Mail list logo