openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pinaki Poddar (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (OPENJPA-798) slices does can not work with sequences properly (no adding objects?)
Date Thu, 26 Feb 2009 17:21:01 GMT

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

Pinaki Poddar resolved OPENJPA-798.
-----------------------------------

    Resolution: Fixed

> slices does can not work with sequences properly (no adding objects?)
> ---------------------------------------------------------------------
>
>                 Key: OPENJPA-798
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-798
>             Project: OpenJPA
>          Issue Type: Bug
>          Components: slice
>    Affects Versions: 2.0.0
>            Reporter: Fernando
>            Assignee: Pinaki Poddar
>            Priority: Critical
>         Attachments: slices-resultset.diff
>
>
> I can't believe that this was a bug that was existent.. but essentially once I got slices
fixed up (see many preceding bugs), it all seems to work fine, except for I can't persist
new objects.  Essentially the DistributedResultSet is very very buggy, and not implemented
properly, so TableJDBCSeq can't do it's job properly ( since it's using the DistributedConnection/PreparedStatement/RestulSet
).
> I just wanted to file the bug to get the word out.. but I basically have to re-write
the whole class, So I guess I'll be submitting a patch tomorrow.
> The first bug is that it will drop out the first row from every ResultSet (one per db).
 It does this because as ResultSets are added, it calls "first", but then most people (and
TableJDBCSeq), assume that it's "beforeFirst", so they call "next" expecting it to move to
"first".  So in TableJDBCSeq, it goes beyond the first row, returning null.. and TableJDBCSeq
thinks that the query actually returned an empty RestultSet and tries to act accordingly...
> and as you review DistributedResultSet, you see lots and lots of bad logic, missing code,
etc.  It's a pretty hacked together implementation of a "Distributed ResultSet".... need a
proper version..

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message