Return-Path: Delivered-To: apmail-incubator-empire-db-dev-archive@minotaur.apache.org Received: (qmail 73008 invoked from network); 11 Oct 2010 06:33:16 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 11 Oct 2010 06:33:16 -0000 Received: (qmail 30770 invoked by uid 500); 11 Oct 2010 06:33:16 -0000 Delivered-To: apmail-incubator-empire-db-dev-archive@incubator.apache.org Received: (qmail 30728 invoked by uid 500); 11 Oct 2010 06:33:15 -0000 Mailing-List: contact empire-db-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: empire-db-dev@incubator.apache.org Delivered-To: mailing list empire-db-dev@incubator.apache.org Received: (qmail 30714 invoked by uid 99); 11 Oct 2010 06:33:14 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 11 Oct 2010 06:33:14 +0000 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests=FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_PASS,T_TO_NO_BRKTS_FREEMAIL X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of francisdb@gmail.com designates 209.85.212.47 as permitted sender) Received: from [209.85.212.47] (HELO mail-vw0-f47.google.com) (209.85.212.47) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 11 Oct 2010 06:33:08 +0000 Received: by vws3 with SMTP id 3so598188vws.6 for ; Sun, 10 Oct 2010 23:32:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:mime-version:received:in-reply-to :references:from:date:message-id:subject:to:content-type :content-transfer-encoding; bh=8RmA8L47cEuusZl4kPJ8IUXYGEsS7RHXqaG09uTLAH0=; b=JzwVVHcLdnz9SxVmkygyzD3RHp7oQOr0f3m8AOQTMwkYv3RxlnymW6F0vemoHQiDjf M9cKQDgT804Wcj4UekIiIwcESBEEFq+8pfr1HkyhgJnIfcbj3aKMGyUbS55BbYYUntXG lggCTZFkxawA3zlYyC7uUVHohGfmfWEsX1LNU= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type:content-transfer-encoding; b=bV4Sc1WbZ8keycHje8SCMG6KdvvKKXk7wfPX+K12T0LhZrwNJE6EUU6IxawMFmfh8k yZOPidknQ7nM2bl7hDdTOVeVPnpfh8nAOWZiAZeZ1Qg50Xs3yNyPTj8t0fCfeDvcHhSX JM12rCkOCfkmgUnEv6ox7/4VSLfoWrPCni3eg= Received: by 10.220.95.204 with SMTP id e12mr1809996vcn.146.1286778764449; Sun, 10 Oct 2010 23:32:44 -0700 (PDT) MIME-Version: 1.0 Received: by 10.220.200.204 with HTTP; Sun, 10 Oct 2010 23:32:24 -0700 (PDT) In-Reply-To: References: <0558C595-598F-4CA5-8082-1BE3277092DC@andreasfink.com> From: Francis De Brabandere Date: Mon, 11 Oct 2010 08:32:24 +0200 Message-ID: Subject: Re: AW: year "functionName" for Oracle To: empire-db-dev@incubator.apache.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-Virus-Checked: Checked by ClamAV on apache.org On Mon, Oct 11, 2010 at 1:19 AM, Marcio Mazza wrote= : > Thanks for the answer, Rainer. > > Maybe I should clarify that the little test was just for > communication, and not to actually test functionality. > > The concern is that the method > =A0String org.apache.empire.db.expr.column.DBFuncExpr.getFunctionName() > =A0is used by > =A0 =A0Element org.apache.empire.db.expr.column.DBAbstractFuncExpr.addXml= (Element > parent, long flags) > =A0 =A0which is used by > =A0 =A0 =A0boolean org.apache.empire.db.DBReader.addColumnDesc(Element pa= rent) > =A0 =A0 =A0which is used by > =A0 =A0 =A0 =A0Document org.apache.empire.db.DBReader.getXmlDocument() > > so maybe we may end up having something unexpected with the XML > generation. I really dunno. I'm still reading the codebase. > > By the way, I really think that the project should have more tests, > some of which would be integration tests, indeed. I agree, feel free to submit some in jira > > > > 2010/10/10 Andreas Fink : >> oracle? never heard of ;-) >> >>> So since I am sure, that there are not many people using Oracle 8 or an= earlier version any more, I will change the default implementation for the= date extraction functions >> [...] >>> compatibility. >>> Does anyone have concerns about this? >> >> +1 for the change >> >> regards, >> andi >> >> >> -- >> web: =A0 http://andreasfink.com/ >> mail: =A0af@andreasfink.com >> mobil: +34 65 1728443 >> tel: =A0 +34 93 2082949 >> >> > --=20 http://www.somatik.be Microsoft gives you windows, Linux gives you the whole house.