Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 341363

Summary:

Add ability to return multiple table columns from SQLQueryHandler

Product:

[Modeling] EMF

Reporter:

Szabolcs Bardy <szbardy>

Component:

cdo.db

Assignee:

Eike Stepper <stepper>

Status:

CLOSED FIXED

QA Contact:

Eike Stepper <stepper>

Severity:

enhancement

Priority:

Keywords:

noteworthy

Version:

4.0

Flags:

stepper: indigo+

Target Milestone:

---

Hardware:

OS:

Windows 7

Whiteboard:

Power to the People

Attachments:

Description	Flags
Proposed patch	stepper: iplog+
Unit test method	stepper: iplog+
Patch v2 (includes tests)	none

Description Szabolcs Bardy

2011-03-30 10:18:24 EDT

Build Identifier: 

At the moment SQL CDO queries return only the first column as a result even if not a CDOObject query is run. If the query is not a CDOObject query, the client is probably interested in all of the queried fields. It would be fine if the query would return the results as an array.

Reproducible: Always

Comment 1 Szabolcs Bardy

2011-03-30 10:21:20 EDT

Created attachment 192193 [details]
Proposed patch

Proposed patch for the SQLQueryHandler class.

Comment 2 Szabolcs Bardy

2011-03-30 10:24:04 EDT

Created attachment 192195 [details]
Unit test method

Test method added to org.eclipse.emf.cdo.tests.db.SQLQueryTest.

Comment 3 Eike Stepper

2011-04-04 11:53:13 EDT

Created attachment 192470 [details]
Patch v2 (includes tests)

Seems to work perfectly. Thanks!

Comment 4 Eike Stepper

2011-04-04 11:54:11 EDT

Szabolcs, please confirm that:

1) The number of lines that you changed is smaller than 250.
2) You are the only author of these changed lines.
3) You apply the EPL to these changed lines.

Comment 5 Szabolcs Bardy

2011-04-04 12:16:17 EDT

(In reply to comment #4)
> Szabolcs, please confirm that:
> 
> 1) The number of lines that you changed is smaller than 250.
> 2) You are the only author of these changed lines.
> 3) You apply the EPL to these changed lines.

Hi Eike,

I can confirm these statements.

Regards,
Szabolcs

Comment 6 Eike Stepper

2011-04-04 21:42:29 EDT

Thank you!

Committed revision 7586

Comment 7 Eike Stepper

2011-06-23 03:38:13 EDT

Available in R20110608-1407