Apache JDO Project - QueryElementHolder patch

This is Interesting: Free IT Magazines  
Home > Archive > Apache JDO Project > November 2005 > QueryElementHolder patch





You are viewing an archived Text-only version of the thread. To view this thread in it's original format and/or if you want to reply to this thread please [click here]

Author QueryElementHolder patch
Michael Watzek

2005-11-03, 5:45 pm

Hi,

I'm sending this patch to jdo-dev because JIRA is not available currently.

JPOX has adapted the syntax of range clauses in single string queries to
match the latest spec. This patch fixes the issue in class
QueryElementHolder. Please review.

Regards,
Michael
--
-------------------------------------------------------------------
Michael Watzek Tech@Spree Engineering GmbH
mailto:mwa.tech@spree.de Buelowstr. 66
Tel.: ++49/30/235 520 36 10783 Berlin - Germany
Fax.: ++49/30/217 520 12 http://www.spree.de/
-------------------------------------------------------------------

Michael Bouschen

2005-11-03, 5:45 pm

Hi Michael,

looks good, check it in.

Regards Michael

> Hi,
>
> I'm sending this patch to jdo-dev because JIRA is not available
> currently.
>
> JPOX has adapted the syntax of range clauses in single string queries
> to match the latest spec. This patch fixes the issue in class
> QueryElementHolder. Please review.
>
> Regards,
> Michael
>
>------------------------------------------------------------------------
>
>Index: test/java/org/apache/jdo/tck/query/QueryElementHolder.java
> ========================================
===========================
>--- test/java/org/apache/jdo/tck/query/QueryElementHolder.java (Revision 330555)
>+++ test/java/org/apache/jdo/tck/query/QueryElementHolder.java (Arbeitskopie)
>@@ -303,10 +303,10 @@
> private String rangeToString() {
> String result = "";
> if (this.fromString != null && this.toString != null) {
>- result = "RANGE " + this.fromString + " TO " + this.toString;
>+ result = "RANGE " + this.fromString + ',' + this.toString;
> }
> else if (this.fromLong != null && this.toLong != null) {
>- result = "RANGE " + this.fromLong + " TO " + this.toLong;
>+ result = "RANGE " + this.fromLong + ',' + this.toLong;
> }
> return result;
> }
>
>



--
Michael Bouschen Tech@Spree Engineering GmbH
mailto:mbo.tech@spree.de http://www.tech.spree.de/
Tel.:++49/30/235 520-33 Buelowstr. 66
Fax.:++49/30/2175 2012 D-10783 Berlin


Sponsored Links






Free braindumps | Software forum | Database administration forum

Copyright 2003 - 2008 webservertalk.com