Re: autocommit feature in pgadmin
От | Sanket Mehta |
---|---|
Тема | Re: autocommit feature in pgadmin |
Дата | |
Msg-id | CA+yw=mMkfzLsyKU84tWd5VBkgpj5ERKmmrfQ8_9i6NK=C_AE6Q@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: autocommit feature in pgadmin (Ashesh Vashi <ashesh.vashi@enterprisedb.com>) |
Ответы |
Re: autocommit feature in pgadmin
|
Список | 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:
and, sysSettings::SetAutoCommit()You should be using the function, you've defined in sysSettings.You've used "settings->Read(wxT("frmQuery/AutoCommit"), &bVal, true)" directly.Hi Sanket,In your patch, I see couple of issues with your patch:
1. In pgadmin/frm/frmQuery.cpp:
i.e.
sysSettings::GetAutoCommit()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.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 commit2. In Query editor -> Query menu -> Auto-commmitBy 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 MehtaSr Software engineerEnterprisedb
Вложения
В списке pgadmin-hackers по дате отправления: