Mailing-List: contact torque-dev-help@db.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Apache Torque Developers List" <torque-dev@db.apache.org>
Received-SPF: pass (athena.apache.org: local policy)
Sensitivity: 
MIME-Version: 1.0
Importance: Normal
In-Reply-To: <4FB0127F.2010807@apache.org>
References: <4FB0127F.2010807@apache.org>,
	<4FAF83D0.2060908@apache.org>,
 <OFE4F1285F.5B88C0CE-ONC12579FD.002E8F0D-C12579FD.002E8F0E@seitenbau.net>
 <OF947F77D1.6968B13C-ONC12579FD.00374E5C-C12579FD.00374E5D@seitenbau.net>
 <4FAFAF57.5080908@apache.org>
Subject: Re: ready for 4.0-alpha1 ?
From: Thomas Fox <Thomas.Fox@seitenbau.net>
To: torque-dev@db.apache.org
Message-ID: 
 <OFA938BE6E.68AA39B8-ONC12579FE.001723E2-C12579FE.001723E3@seitenbau.net>
Date: Mon, 14 May 2012 06:12:45 +0200
Content-type: text/plain; charset=US-ASCII


>>> Also, I have forgotten one thing: The BaseManager interface does not
look
>>> very typesafe to me, it uses a lot the Persistent interface, and casts
are
>>> needed to the correct OM type. Maybe this could be improved by
generifying
>>> the BaseManager class... But then I have never used managers, I have no
>>> idea what a good interface is. Anybody interested there ?
>>
>> Well, this obviously is my playground. However, I'm currently
>> generifying JCS, so don't expect anything soon. Maybe I can squeeze it
in.

>Ok, in an attack of sheer madness I made an attempt to solve this. It
>happened to be code deletion, mostly, given the features of JDK 5. I'd
>ask for close inspection as I'm not used to the code base anymore.

I am not very familiar to the manager code, but from reading the changes
looked good to me. Jenkins says the tests ran ok (the hsqldb test runs the
managers test)

>A couple of comments, though:
>- Creating assemblies on every mvn install seems to be overkill to me. I
>suggest to create assemblies only on release

I'll look at the commons parent pom as you suggested and try to fix this.

>- The torque-test with MySQL failed on me because the table OPTION could
>not be created. I don't know if this is really important, but
>identifiers an be quoted with `backticks` in MySQL, which is easily
>added in the templates.

Hm, last time I ran the mysql tests they were ok although a couple of table
could not be created. Does it say "Build successful" in the end ?
I created a jira ticket for the quoting on creation issue:
https://issues.apache.org/jira/browse/TORQUE-203
Not sure, though, what happens if we access these tables from within
Torque...

>- Debugging the template code is a PITA.

Hm, always has been, at least for me... Not being a velocity guy, I always
run the test project and checked the outcome...
What exactly is the problem ? Templates organization(finding things) ?

An then there's the change log (changes.cml via jira) question still open.
Do we want to continue use changes.xml in the future or do we want to use
the jira change log... ?

   Thomas


---------------------------------------------------------------------
To unsubscribe, e-mail: torque-dev-unsubscribe@db.apache.org
For additional commands, e-mail: torque-dev-help@db.apache.org