From dev-return-7341-apmail-continuum-dev-archive=continuum.apache.org@continuum.apache.org Thu Sep 11 20:40:28 2008 Return-Path: Delivered-To: apmail-continuum-dev-archive@www.apache.org Received: (qmail 75691 invoked from network); 11 Sep 2008 20:40:28 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 11 Sep 2008 20:40:28 -0000 Received: (qmail 87838 invoked by uid 500); 11 Sep 2008 20:40:24 -0000 Delivered-To: apmail-continuum-dev-archive@continuum.apache.org Received: (qmail 87797 invoked by uid 500); 11 Sep 2008 20:40:24 -0000 Mailing-List: contact dev-help@continuum.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@continuum.apache.org Delivered-To: mailing list dev@continuum.apache.org Received: (qmail 87786 invoked by uid 99); 11 Sep 2008 20:40:24 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 11 Sep 2008 13:40:24 -0700 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of oliver.lamy@gmail.com designates 209.85.132.241 as permitted sender) Received: from [209.85.132.241] (HELO an-out-0708.google.com) (209.85.132.241) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 11 Sep 2008 20:39:22 +0000 Received: by an-out-0708.google.com with SMTP id d30so66443and.30 for ; Thu, 11 Sep 2008 13:39:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:sender :to:subject:in-reply-to:mime-version:content-type :content-transfer-encoding:content-disposition:references :x-google-sender-auth; bh=8Dbr5Aww65GaEG6gFxCXzKiBY8kT4e9+wfBc78f6lVU=; b=KDQiRn/LpjL6WsxXWcCcQ7GDPYoLjqh2eqr8tzI6RKLHEirbDa0cwdNBhbzG/JU647 Jb2Ez0nxnkpmzDzEMbSTUL58b9DdAH+vr+WxeKvpaN2S0LFjt8sIci8S6qvXmanTHR3l RjcoQJDMciSDC9JiWVur8bwXZc54KQwmJfWNM= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:sender:to:subject:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references:x-google-sender-auth; b=pE/R89PfeOdBI9DXZyeEuQgBBkB+C94lJcsYt9od6nd56Ol2h7EmFo9r0rF6kV7n7o jMfwH6GP81CUHAiMk0xd1G09NWBLgjAjXlyIISrkT6gwbLvZPJyZFe5htcprVluInIAh dW7okRH18IDnNUbqnKgVZ/8rales1msQ/1Jok= Received: by 10.187.221.20 with SMTP id y20mr453290faq.107.1221165575483; Thu, 11 Sep 2008 13:39:35 -0700 (PDT) Received: by 10.187.183.8 with HTTP; Thu, 11 Sep 2008 13:39:35 -0700 (PDT) Message-ID: <9948cb690809111339y3a721d79p8de7910c73aa69c7@mail.gmail.com> Date: Thu, 11 Sep 2008 22:39:35 +0200 From: "Olivier Lamy" Sender: oliver.lamy@gmail.com To: dev@continuum.apache.org Subject: Re: Continuum 1.2 and SQL Server In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <9948cb690809100004y4068f4er119e15c527d65cf8@mail.gmail.com> <9948cb690809100037t3a2cac85vc3d2694fe4725e52@mail.gmail.com> <007601c91329$62b49b00$281dd100$@com> <9948cb690809100848u57dfdddq854d55788f5787b7@mail.gmail.com> <023701c9140e$b92e80a0$2b8b81e0$@com> <004a01c91420$2b1b53d0$8151fb70$@com> X-Google-Sender-Auth: 2e24d509c121f682 X-Virus-Checked: Checked by ClamAV on apache.org Jimmy, Patch applied. Thanks ! -- Olivier PS : please next time can you svn diff to generate the patch file. (http://continuum.apache.org/development/guide-continuum-development.html#Creating_and_submitting_a_patch) 2008/9/11 Emmanuel Venisse : > 1- checkout continuum2- > modify continuum-model/src/main/resources/package-mssql.orm to use 4000 > 3- rebuild Continuum > 4- check it is ok with your sql server > 5- attach your patch on a jira issue > > Emmanuel > > On Thu, Sep 11, 2008 at 5:07 PM, Jimmy Conway wrote: > >> The documentation says - 4000 [1] >> >> What to do? >> >> [1] http://msdn.microsoft.com/en-us/library/ms186939.aspx >> >> Jim >> >> -----Original Message----- >> From: Emmanuel Venisse [mailto:emmanuel.venisse@gmail.com] >> Sent: Thursday, September 11, 2008 4:45 PM >> To: dev@continuum.apache.org >> Subject: Re: Continuum 1.2 and SQL Server >> >> Hmm. >> 4000, 8000 are numbers sent by sql server. >> Maybe 8000 was a max number for all data types without check the filed >> type. >> On the second step (4000), you don't exceed the max, so it check the field >> type. >> >> Can you check in the sql server doc what is the max allowed for a nvarchar? >> >> Emmanuel >> >> On Thu, Sep 11, 2008 at 3:02 PM, Jimmy Conway wrote: >> >> > Oliver, >> > >> > I did this and now I'm getting this message in continuum.log: >> > >> > 2008-09-11 08:46:55,135 [WrapperSimpleAppMain] ERROR JPOX.RDBMS - Error >> > thrown executing CREATE TABLE CHANGESET >> > ( >> > CHANGESET_ID bigint NOT NULL, >> > AUTHOR nvarchar(255) NULL, >> > CHANGECOMMENT nvarchar(8000) NULL, >> > CHANGEDATE bigint NOT NULL, >> > ID nvarchar(255) NULL, >> > MODEL_ENCODING nvarchar(255) NULL, >> > CHANGES_SCMRESULT_ID_OID bigint NULL, >> > CHANGES_INTEGER_IDX int NULL >> > ) : The size (8000) given to the parameter 'CHANGECOMMENT' exceeds the >> > maximum allowed (4000). >> > com.microsoft.sqlserver.jdbc.SQLServerException: The size (8000) given to >> > the parameter 'CHANGECOMMENT' exceeds the maximum allowed (4000). >> > >> > Why it was 8000 before, now it's 4000... Any ideas? >> > >> > Thanks! >> > >> > Jim >> > >> > -----Original Message----- >> > From: oliver.lamy@gmail.com [mailto:oliver.lamy@gmail.com] On Behalf Of >> > Olivier Lamy >> > Sent: Wednesday, September 10, 2008 5:48 PM >> > To: dev@continuum.apache.org >> > Subject: Re: Continuum 1.2 and SQL Server >> > >> > Hi, >> > IMHO this cannot be included in the distribution by default as it's a >> > specific database vendor issue. >> > In the webapp, you will find the file in >> > WEB-INF/classes/META-INF/plexus/application.xml. >> > You will find something like : >> > I'm sure this will help :-) >> > But sure this need to be added in the documenation. >> > >> > -- >> > Olivier >> > >> > 2008/9/10 Jimmy Conway : >> > > Olivier, >> > > >> > > I have SQL Server and I have this problem (I reported it in users@). >> > > >> > > Please, let me know how to test it, I'm ready. I use the latest code >> from >> > > /trunk. So, as soon as you raise a flag, I will compile and try to run >> > with >> > > my SQL Server. >> > > >> > > Thanks. >> > > >> > > Jim >> > > >> > > >> > > -----Original Message----- >> > > From: oliver.lamy@gmail.com [mailto:oliver.lamy@gmail.com] On Behalf >> Of >> > > Olivier Lamy >> > > Sent: Wednesday, September 10, 2008 9:37 AM >> > > To: dev@continuum.apache.org >> > > Subject: Re: Continuum 1.2 and SQL Server >> > > >> > > Great. >> > > But why it doesn't work and how to enable this ? >> > > >> > > Sorry I don't have a mssql server here ;-) >> > > >> > > -- >> > > Olivier >> > > >> > > 2008/9/10 Emmanuel Venisse : >> > >> The size (8000) is already define for mssql: >> > >> >> > > >> > >> > >> >> https://svn.apache.org/repos/asf/continuum/trunk/continuum-model/src/main/re >> > > sources/package-mssql.orm >> > >> >> > >> Emmanuel >> > >> >> > >> On Wed, Sep 10, 2008 at 9:04 AM, Olivier Lamy >> wrote: >> > >> >> > >>> arghhhh :-))) >> > >>> >> > >>> It's due to http://jira.codehaus.org/browse/CONTINUUM-1688. >> > >>> >> > >>> We can certainly reduce the size to 8000. >> > >>> I will reopen the issue. >> > >>> IMHO, we should control it and why not reduce the String if it's too >> > long >> > >>> >> > >>> -- >> > >>> Olivier >> > >>> PS : olamy complains about database vendor restriction ;-) >> > >>> >> > >>> 2008/9/10 Brett Porter : >> > >>> > Sorry for finding more issues :) Since this one is a regression I >> > > thought >> > >>> it >> > >>> > might require some attention. >> > >>> > >> > >>> > I saw on the users@ list that someone had a problem using SQL >> Server >> > >>> because >> > >>> > of the length of changeComment being increased to 8192 (> 8000). I >> > > think >> > >>> > this was previously shorter - is that right? >> > >>> > >> > >>> > If so, should it be reduced by this small amount? >> > >>> > >> > >>> > I'm still getting to test the database upgrade... once done and >> > >>> documented >> > >>> > I'm happy with the release. >> > >>> > >> > >>> > - Brett >> > >>> > >> > >>> > -- >> > >>> > Brett Porter >> > >>> > brett@apache.org >> > >>> > http://blogs.exist.com/bporter/ >> > >>> > >> > >>> > >> > >>> >> > >> >> > > >> > > __________ Information from ESET NOD32 Antivirus, version of virus >> > signature >> > > database 3430 (20080910) __________ >> > > >> > > The message was checked by ESET NOD32 Antivirus. >> > > >> > > http://www.eset.com >> > > >> > > >> > > >> > > __________ Information from ESET NOD32 Antivirus, version of virus >> > signature >> > > database 3430 (20080910) __________ >> > > >> > > The message was checked by ESET NOD32 Antivirus. >> > > >> > > http://www.eset.com >> > > >> > > >> > > >> > >> > __________ Information from ESET NOD32 Antivirus, version of virus >> > signature >> > database 3432 (20080910) __________ >> > >> > The message was checked by ESET NOD32 Antivirus. >> > >> > http://www.eset.com >> > >> > >> > >> > __________ Information from ESET NOD32 Antivirus, version of virus >> > signature >> > database 3434 (20080911) __________ >> > >> > The message was checked by ESET NOD32 Antivirus. >> > >> > http://www.eset.com >> > >> > >> > >> >> >> >> __________ Information from ESET NOD32 Antivirus, version of virus >> signature >> database 3434 (20080911) __________ >> >> The message was checked by ESET NOD32 Antivirus. >> >> http://www.eset.com >> >> >