Re: Review Request 306: RM 33614 - Autocommit on/off option in PEM client

From: Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com>
To: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Review Request 306: RM 33614 - Autocommit on/off option in PEM client
Date: 2015-03-24 11:57:53
Message-ID: CA+yw=mOvwN6+xv5YE3+=-PQ6+5ra+xSVGSVcQ7NW_f9Suf+waw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

I have developed the *Autocommit* feature in pgadmin and its patch is
attached with this mail.
Please do review and commit it.

Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb

On Tue, Mar 17, 2015 at 2:01 AM, Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com
> wrote:

> This is an automatically generated e-mail. To reply, visit:
> https://scm.enterprisedb.com/reviews/r/306/
> Review request for Postgres Enterprise Manager, Akshay Joshi and Ashesh
> Vashi.
> By Sanket Mehta.
> *Repository: * PEM
> Description
>
> In Autocommit on/off option in pem client:
>
> "Enable autocommit" option is provided in pem client at 2 places:
>
> 1. pem client -> File menu -> Options -> Query Editor -> Enable AUTO COMMIT
> 2. sql query tool -> Query menu -> Auto-commit
>
> User can select/deselect its value from any of the above 2 places and it will be reflected to other one.
>
> In any query tool session, once user uncheck this checkbox, depending upon postgreSQL standard for executing the query it will execute "BEGIN;" statement first at the time of executing the 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;
>
>
> Testing
>
> - setting autocommit on/off from client window.
> - setting autocommit on/off from query tool.
> - Setting the autocommit mode on/off in database
>
> Diffs
>
> - pgadmin/ui/frmOptions.xrc (a57b0ce)
> - pgadmin/include/frm/menu.h (fb20b60)
> - pgadmin/include/frm/frmQuery.h (c698370)
> - pgadmin/frm/frmQuery.cpp (23ec45b)
> - pgadmin/frm/frmOptions.cpp (4b3b1f1)
> - docs/en_US/options-query_tool.rst (0b557e7)
>
> View Diff <https://scm.enterprisedb.com/reviews/r/306/diff/>
>

Attachment Content-Type Size
RM_33614v3.patch application/octet-stream 99.6 KB

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Sanket Mehta 2015-03-24 12:31:23 autocommit feature in pgadmin
Previous Message Ashesh Vashi 2015-03-23 08:58:57 Re: PgAgent Patch