Re: [datanucleus/datanucleus-api-jdo] JDOQL: default query result does not work with executeResultList (#52)

[Michael Bouschen <mi...@akquinet.de>]

Hi Andy,

> Why are you calling setResultClass() (legacy API) when using
> executeResultList(resultClass) which tells it the result class ? [not
> that this affects the result, but I don't get the mix of legacy with
> generics APIs]
>
yes, calling setResultClass is not necessary for this test. I just
called it to see whether it makes any difference with respect to the issue.
>
> This issue simply comes down to whether it is reasonable to call
> executeResultList when there is no result defined (i.e returning a
> candidate). Current implementation assumes that it isn't reasonable
> (since why return a candidate in a result object?). Could easily
> change, but anyway ...
>

The query is not just returning the candidate instances, they should be
wrapped into PersonWrapper instances. In order to specify that the
result should be wrapped I have to call executeResultList passing the
wrapper class (or call setResultClass with the result class). The JDO
spec has two similar sample queries in chapter 14.10.15 and chapter
14.10.16 that both store the result in a user defined class and both
rely on the default setting for Query.setResult.

Regards Michael
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/datanucleus/datanucleus-api-jdo/issues/52#issuecomment-304464716>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AbdDGASaJbkgEXyhatkdSJAmdNhwFFIjks5r-FkIgaJpZM4NoZIP>.
>


-- 
Michael Bouschen
akquinet tech@spree GmbH
Bülowstraße 66 • D-10783 Berlin
Tel:   +49 30 235520-33
Fax:  +49 30 217520-12

E-Mail: michael.bouschen@akquinet.de <ma...@akquinet.de>
Web:   www.akquinet.de <http://www.akquinet.de/>

Geschäftsführung: Martin Weber, Dr. Torsten Fink
Amtsgericht Berlin HRB 86780 • USt.-Id. Nr.: DE 225 964 680

[Facebook] <http://www.facebook.com/akquinet>  [XING]
<https://www.xing.com/companies/akquinetag>  [G+]
<https://plus.google.com/b/111054946250796705170/+akquinet/posts> 
[LinkedIn] <https://www.linkedin.com/company/akquinet-ag>  [Twitter]
<https://twitter.com/akquinet>