[ 
https://issues.apache.org/jira/browse/DERBY-4687?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Kathey Marsden updated DERBY-4687:
----------------------------------

       Urgency: Low
        Labels: Connection client derby_triage10_8 rollback  (was: Connection 
client rollback)
    Issue Type: Improvement  (was: Bug)

Triage for 10.8. Marked urgency low since the commit case is usually most 
important for performance.  Changing from bug to improvement.

> Avoid unnecessary round-trip for rollback in the client driver
> --------------------------------------------------------------
>
>                 Key: DERBY-4687
>                 URL: https://issues.apache.org/jira/browse/DERBY-4687
>             Project: Derby
>          Issue Type: Improvement
>          Components: Network Client
>    Affects Versions: 10.6.1.0
>            Reporter: Lily Wei
>            Assignee: Lily Wei
>            Priority: Minor
>              Labels: Connection, client, derby_triage10_8, rollback
>
> Per fixing DERBY-4653, the same issue is happening for Connection.rollback().
> The methods Connection.rollback() in the client driver cause a round-trip to 
> the server even if the rollback is unnecessary (i.e. there is nothing to roll 
> back).
> Comments suggest (see below) that this can be optimized, such that the 
> commands are flowed to the server only when required. It can be seen that 
> this optimization has been used other places in the client driver. Never the 
> less, it must be checked that this optimization doesn't have side-effects and 
> does not create regression.
>     // Even if we're not in a transaction, all open result sets will be 
> closed.
>     // So we could probably just return if we're not in a transaction
>     // using the following code:
>     //     if (!this.inUnitOfWork)
>     //       return;
> The protocol tests should be added too.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to