[ 
https://issues.apache.org/jira/browse/RANGER-4104?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17692267#comment-17692267
 ] 

Andrew Luo commented on RANGER-4104:
------------------------------------

The annotation on the entity class affects how the ResultSet from JDBC is 
converted to an object, but when calling setParameter we still have to specify 
TemporalType.DATE.  Otherwise there would be no need for setParameter to take 
that parameter if the entity column annotation was sufficient (from a design 
perspective, I don't know exactly why this is but I suspect it's might to allow 
compatible types to be compared, for example, Date and Time)

> XXAuthSessionDao.getRecentAuthFailureCountByLoginId produces incorrect SQL 
> code
> -------------------------------------------------------------------------------
>
>                 Key: RANGER-4104
>                 URL: https://issues.apache.org/jira/browse/RANGER-4104
>             Project: Ranger
>          Issue Type: Bug
>          Components: admin
>    Affects Versions: 3.0.0, 2.4.0
>            Reporter: Andrew Luo
>            Priority: Major
>         Attachments: 
> 0001-Fix-type-of-authWindowStartTime-parameter-in-XXAuthS.patch
>
>
> XXAuthSessionDao.getRecentAuthFailureCountByLoginId produces incorrect SQL 
> code due to an error in how the authWindowStartTime Date parameter is bound.
> It is currently bound with setParameter("authWindowStartTime", 
> authWindowStartTime) however, [JPA 2.2 Specification Section 
> 11.1.53|https://download.oracle.com/otn-pub/jcp/persistence-2_2-mrel-spec/JavaPersistence.pdf]
>  says that it should be bound by specifying an additional parameter 
> TemporalType.DATE.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to