From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: autocommit feature in pgadmin |
Date: | 2015-03-31 16:13:32 |
Message-ID: | CAG7mmozcQ1guz2y5hBDNWfNoihw4Z2yW418dNvjR_2N4YWSw3A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Sanket,
In your patch, I see couple of issues with your patch:
1. In pgadmin/frm/frmQuery.cpp:
You've used "settings->Read(wxT("frmQuery/AutoCommit"), &bVal, true)"
directly.
You should be using the function, you've defined in sysSettings.
i.e.
sysSettings::GetAutoCommit()
and, sysSettings::SetAutoCommit()
Please follow the correct naming convention for the
frmQuery::CommandNoBegin(...) function.
Function must not start with a capital letter for a regular function (we do
declare/define only the event functions with capital letters).
2. In pgadmin/ui/frmOptions.xrc:
A lot of changes are in this patch, which not required for this feature.
--
Thanks & Regards,
Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company
<http://www.enterprisedb.com>
*http://www.linkedin.com/in/asheshvashi*
<http://www.linkedin.com/in/asheshvashi>
On Tue, Mar 24, 2015 at 6:01 PM, Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com
> wrote:
> Hi,
>
> Below is the description of the autocommit feature implementation in
> pgadmin:
>
> This feature is only applicable in query editor.
>
> Autocommit can be set on/off from 2 places.
>
> 1. pgadmin main browser -> File menu -> Options -> Query tool -> Query
> editor -> Enable Auto commit
> 2. In Query editor -> Query menu -> Auto-commmit
>
> By default auto commit will be enabled in pgadmin.
>
>
> In any query editor session, once user uncheck this autocommit checkbox,
> depending upon postgreSQL standard for executing the query it will execute
> "BEGIN;" statement before executing the actual query and thus setting the
> autocommit off for that session.
>
> User can set the autocommit on at any time by selecting the checkbox from
> above mentioned places.
> But it will be active only after user has completed the transaction
> block(started by BEGIN as mentioned above) by END or ROLLBACK or COMMIT;
>
> Let me know in case of any queries.
>
> Regards,
> Sanket Mehta
> Sr Software engineer
> Enterprisedb
>
From | Date | Subject | |
---|---|---|---|
Next Message | Sanket Mehta | 2015-04-01 09:01:21 | Re: autocommit feature in pgadmin |
Previous Message | Ashesh Vashi | 2015-03-31 12:56:14 | Re: PATCH: pgagent fix for 'Caught unhandled unknown exception; terminating' bug in SQL steps |