| time |
nick |
message |
08:10 |
<ralf> |
hi udai |
08:18 |
<Udai> |
hi ralf |
08:20 |
<Udai> |
I have few things to ask and tell you |
08:20 |
<Udai> |
about temporalType |
08:22 |
<Udai> |
as DateExpression, TimeExpression and TimestampExpression are the temporalType |
08:22 |
<Udai> |
for Condition equal(final TemporalType temporalType, final Date value) and Condition equal(final TemporalType temporalType, final Calendar value) { |
08:23 |
<Udai> |
in implementation I will be using java.sql.Date, Time and Timestamp |
08:25 |
<Udai> |
java.util.Date and Calendar are more general datatypes |
08:26 |
<Udai> |
I will be breaking them acc to tempralType during the implementation |
08:26 |
<Udai> |
I have renamed property start to index with getter and setters in Substring function according to the Expression interface substring defination |
08:27 |
<Udai> |
I have renamed properties start to index and search to value with getter and setters in Locate function according to the Expression interface Locate defination |
08:27 |
<Udai> |
about the enum OrderDirective |
08:29 |
<Udai> |
ASCENDING should be ACS and DESCENDING should be DESC so that they can be converted into toString method directly |
08:30 |
<Udai> |
I will mention these points when I would attach the new patch that will contain whole QO package |
08:31 |
<Udai> |
do you have any comments on these right now?? |
08:37 |
<Udai> |
will be out for lunch..... |
08:37 |
<Udai> |
I will be back soon.... |
08:49 |
<ralf> |
okay, sounds fine by me |
10:22 |
<ralf> |
also out for lunch |
17:36 |
<ralf> |
udai: will be back in about an hour to look at parts of your work |
17:37 |
<ralf> |
if you will not be online at that time please send me a mail what you prefer where i should start |
17:38 |
<ralf> |
for your information. I'll be offline tomorrow but plan to continue review of your work at sunday |
17:38 |
<ralf> |
hope to see you later |
19:15 |
<Udai> |
ralf?? |
19:23 |
<ralf> |
hi udai |
19:23 |
<ralf> |
i'm back now |
19:23 |
<Udai> |
hi |
19:23 |
<ralf> |
how are things going |
19:23 |
<Udai> |
glad to see you back |
19:23 |
<Udai> |
just fine |
19:23 |
<Udai> |
I hope you got the mail |
19:24 |
<ralf> |
just checking my mails |
19:24 |
<Udai> |
k |
19:25 |
<ralf> |
okay. i'll look at 2442 |
19:26 |
<Udai> |
and for methods like equal(final TemporalType temporalType, final Date value) |
19:27 |
<Udai> |
here you have took value as java.util.Date |
19:27 |
<Udai> |
and java.util.Calendar; |
19:27 |
<ralf> |
that's according to EJB specification |
19:28 |
<Udai> |
where? |
19:28 |
<Udai> |
can you please point me there |
19:28 |
<ralf> |
at query interface of EJB you have several methods to bind values to parameters |
19:29 |
<ralf> |
and they use the same TemporalType and Date or Calendar |
19:29 |
<Udai> |
k |
19:29 |
<ralf> |
i'll check |
19:29 |
<Udai> |
cause I was considering the string format for date, time and timestamp |
19:30 |
<ralf> |
page 66 |
19:30 |
<Udai> |
okay |
19:32 |
<ralf> |
it's always difficult to find the best way to work with dates |
19:32 |
<Udai> |
k then I will workout the implementation acc to them |
19:33 |
<Udai> |
I will format the Date to string acc ot our specification |
19:33 |
<Udai> |
same for others |
19:33 |
<ralf> |
i see 3 options |
19:34 |
<Udai> |
when converting them toString |
19:34 |
<ralf> |
use java.sql types or java.util.Date or Calendar to store them internally |
19:35 |
<ralf> |
while java.sql types and java.util.Date are good for transforming them to string |
19:35 |
<ralf> |
has Calendar advantages when we need to compare date values |
19:36 |
<Udai> |
ya |
19:37 |
<ralf> |
if we use java.util.Date we save quite some code as we can move property and getters/setters into an abstract class |
19:37 |
<Udai> |
but suppose if user need to set the Date |
19:37 |
<Udai> |
then what input will he give |
19:38 |
<Udai> |
a string |
19:38 |
<Udai> |
then we can convert that into any type |
19:39 |
<ralf> |
user has to use SelectQuery interface to create a new DateExpression |
19:39 |
<Udai> |
so we provide him like |
19:39 |
<ralf> |
we do not plan that he uses the implementations |
19:40 |
<Udai> |
DateExpression.setDate(Date) |
19:41 |
<Udai> |
if he just want to DateExpression.setDate((String)"yyyymmdd") |
19:41 |
<ralf> |
as far as i recall at the moment there is a createDate(TemporalType, java.util.Date) method at Select interface |
19:42 |
<ralf> |
no, we do not provide such methods |
19:42 |
<Udai> |
are you sure? |
19:42 |
<ralf> |
he can use DateFormat to parse string into Date himself |
19:42 |
<Udai> |
k |
19:43 |
<Udai> |
I don't see this createDate method I think I will include it |
19:43 |
<ralf> |
problem is that we could only support a fixed format for parsing dates |
19:44 |
<Udai> |
yes you are right |
19:44 |
<ralf> |
so its better to keep such logic outside |
19:44 |
<Udai> |
the createDate method need to be added in SelectQuery interface |
19:44 |
<Udai> |
will do that |
19:45 |
<ralf> |
i defined it somewhere |
19:45 |
<ralf> |
maybe its in QueryFactory |
19:46 |
<Udai> |
okay I will see that |
19:46 |
<ralf> |
i have not opened eclipse yet to check myself |
19:46 |
<ralf> |
but i'm sure it is defined somewhere |
19:46 |
<Udai> |
no problem |
19:47 |
<Udai> |
you just open the last patch and search "createDate" |
19:47 |
<ralf> |
maybe you are right that it is better to use java.sql.Date at DateExpression |
19:47 |
<ralf> |
as this gives us more type savety |
19:48 |
<Udai> |
I think I will give some more reasons why we can use which |
19:48 |
<Udai> |
as I have to do a little bit research on them |
19:48 |
<Udai> |
and the EJB implementation |
19:49 |
<ralf> |
okay |
19:49 |
<Udai> |
so we meet on sunday |
19:49 |
<ralf> |
yes |
19:49 |
<Udai> |
so I guess I will go to bed now |
19:50 |
<Udai> |
I am missing my breakfast ;) |
19:50 |
<ralf> |
i will help a friend with repairing some things at his house tomorrow |
19:50 |
<ralf> |
and in the evening will ahve a small barbecue party |
19:50 |
<Udai> |
okay :) |
19:50 |
<ralf> |
;-) |
19:50 |
<ralf> |
see you on sunday |
19:50 |
<Udai> |
enjoy |
19:50 |
<Udai> |
see you |
19:50 |
<Udai> |
bbye |