Re: autocommit feature in pgadmin - Mailing list pgadmin-hackers

From Sanket Mehta
Subject Re: autocommit feature in pgadmin
Date
Msg-id CA+yw=mMkfzLsyKU84tWd5VBkgpj5ERKmmrfQ8_9i6NK=C_AE6Q@mail.gmail.com
Whole thread Raw
In response to Re: autocommit feature in pgadmin  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Responses Re: autocommit feature in pgadmin
List pgadmin-hackers
Hi,

I have made all the changes mentioned in below mail.
Along with that I have changed the code to read autoRollBack value in pgadmin/frm/frmQuery.cpp file.

previously it was used  "settings->Read(wxT("frmQuery/AutoRollBack"), &bVal, true)" directly.

Now I have changed it to settings->GetAutoRollBack() which is defined in sysSettings class.

I have attached the new patch with this mail.
Please review it and do the needful.



Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Tue, Mar 31, 2015 at 9:43 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
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.linkedin.com/in/asheshvashi


On Tue, Mar 24, 2015 at 6:01 PM, Sanket Mehta <sanket.mehta@enterprisedb.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


Attachment

pgadmin-hackers by date:

Previous
From: Ashesh Vashi
Date:
Subject: Re: autocommit feature in pgadmin
Next
From: Ashesh Vashi
Date:
Subject: pgAgent commit: Fixed a bug in DBconn::GetLastError() function.