[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [eclipselink-dev] createNamedQuery("non-existing")
|
My vote is to throw IllegalArgumentException when calling createNamedQuery with
null, but to make the change in the trunk stream so it will not be introduced in
a patch.
-Tom
Goerler, Adrian wrote:
Hi,
I am just having a look at the em.createNamedQuery method invoked with
nasty parameters.
If I am invoking em.createNamedQuery with a non-existing query name, I
am getting a query that throws an IllegalArgumentException upon execution.
If I am invoking it with null, I am getting an query that throws a NPE
upon execution.
But the IllegalArgumentException is not thrown immediately.
The API states
* *@throws* IllegalArgumentException if a query has not been
* defined with the given name or if the query string is
* found to be invalid
*/
*public* Query createNamedQuery(String name);
So we could throw an IllegalArgumentException in both cases. However,
doing so would strictly be incompatible as there might be coding out
there that creates illegal named queries but never executes them.
What do you think? Should we do something about this or leave it as it is?
-Adrian
------------------------------------------------------------------------
_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev