Re: svn commit: r935207 - in /cayenne/main/trunk: docs/doc/src/main/resources/ framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/query/ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/query/

From: Andrus Adamchik (andru..bjectstyle.org)
Date: Sun Apr 18 2010 - 00:55:57 EDT

  • Next message: Evgeny Ryabitskiy: "Re: svn commit: r935207 - in /cayenne/main/trunk: docs/doc/src/main/resources/ framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/query/ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/query/"

    On Apr 18, 2010, at 12:46 AM, evgen..pache.org wrote:

    > public void setResult(SQLResult resultSet) {
    > + setFetchingDataRows(false); // turn off mapping to
    > DataRows, use explicit
    > this.result = resultSet;
    > }

    Implicit flipping of the DataRows flag outside constructor seems
    counterintuitive. Also SQLResult is not equal to an object fetch. So I
    think this is wrong.

    Andrus

    This archive was generated by hypermail 2.0.0 : Sun Apr 18 2010 - 00:56:40 EDT