[GitHub] [cayenne] johnthuss commented on issue #382: Cayenne 4.2.M1 testing

[GitBox <gi...@apache.org>]

johnthuss commented on issue #382: Cayenne 4.2.M1 testing
URL: https://github.com/apache/cayenne/pull/382#issuecomment-489763663
 
 
   Thanks, except for the class loading issue I posted a pull request for, this works. The other issues that still need to be addressed are these, especially #3:
   
   2) I need to have a method like PropertyUtils.propertyTypeDefinition exposed so I can utilize it in my templates. The refactoring to move logic into PropertyUtils makes the templates clean and simple, but users need access to smaller pieces as well.
   
   3) Using "between" expressions on Joda time DateProperty instances fails. This is a bug that still needs to be fixed. I don't know where to start on this one. The stack trace is below, or can you run the failing test I pushed in the pull request:
   Caused by: org.postgresql.util.PSQLException: Can't infer the SQL type to use for an instance of org.joda.time.DateTime. Use setObject() with an explicit Types value to specify the type to use.
   	at org.postgresql.jdbc.PgPreparedStatement.setObject(PgPreparedStatement.java:949)
   	at com.zaxxer.hikari.pool.HikariProxyPreparedStatement.setObject(HikariProxyPreparedStatement.java)
   	at org.apache.cayenne.access.jdbc.SelectAction.bind(SelectAction.java:60)
   	at org.apache.cayenne.access.jdbc.SelectAction.performAction(SelectAction.java:93)
   	at org.apache.cayenne.access.DataNodeQueryAction.runQuery(DataNodeQueryAction.java:97)
   	at org.apache.cayenne.access.DataNode.performQueries(DataNode.java:273)
   	... 46 more
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services