You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@eagle.apache.org by Jashchahal <gi...@git.apache.org> on 2016/05/04 00:01:39 UTC

[GitHub] incubator-eagle pull request: EAGLE-291 : JDBC: Update transaction...

GitHub user Jashchahal opened a pull request:

    https://github.com/apache/incubator-eagle/pull/174

    EAGLE-291 : JDBC: Update transactions fail in PostgreSQL

    The transactions in Postgres was failing due to insert fail transaction already in progress and we were trying to update if insert fails. Need to rollback the failed insert transaction and then continue with update transaction. Postgres is more strict as compared to MySql, so we need save point so that we can roll it back, if required.
    Had to revert changes for EAGLE-270, as @haoch mentioned that we can't compare between mediumtexts, it need to be varchar.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Jashchahal/incubator-eagle master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-eagle/pull/174.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #174
    
----
commit 002088db65194e6cafc2b637d7aaef89708f4225
Author: Jaspaul <ja...@gmail.com>
Date:   2016-04-06T00:32:26Z

    EAGLE-213 Updates fail for MySql
    There was a check based on the exception message, which was not correct
    for MySql. Changed it to more generic, comparing with "duplicate", it will handle
    more broader range now.

commit 085d6b31075f4c9c2a00a071995b65e6ff67ee52
Author: Jaspaul <ja...@gmail.com>
Date:   2016-04-18T17:36:43Z

    Merge remote-tracking branch 'upstream/master'

commit 9c9c1ae156b9cf0aedf6e59730d1635b7e1b0b6b
Author: Jaspaul <ja...@gmail.com>
Date:   2016-04-18T17:56:55Z

    [EAGLE-270] JDBC: Create table fail for some of the tables

commit 3ed5a8dfc68e353b39b05c9a67fa19e56cc3a9e4
Author: Jaspaul <ja...@gmail.com>
Date:   2016-05-03T21:20:25Z

    Merge remote-tracking branch 'upstream/master'

commit 85529dfd7023ba7632123baa13314713605ab480
Author: Jaspaul <ja...@gmail.com>
Date:   2016-05-03T21:43:04Z

    EAGLE-291 : JDBC: Update transactions fail in PostgreSQL

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-291 : JDBC: Update transaction...

Posted by Jashchahal <gi...@git.apache.org>.
Github user Jashchahal commented on the pull request:

    https://github.com/apache/incubator-eagle/pull/174#issuecomment-220731279
  
    closing this Pul request , this is causing issues, will open new one


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-eagle pull request: EAGLE-291 : JDBC: Update transaction...

Posted by Jashchahal <gi...@git.apache.org>.
Github user Jashchahal closed the pull request at:

    https://github.com/apache/incubator-eagle/pull/174


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---