Bug 341363 - Add ability to return multiple table columns from SQLQueryHandler
Summary: Add ability to return multiple table columns from SQLQueryHandler
Status: CLOSED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: cdo.db (show other bugs)
Version: 4.0   Edit
Hardware: PC Windows 7
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Eike Stepper CLA
QA Contact: Eike Stepper CLA
URL:
Whiteboard: Power to the People
Keywords: noteworthy
Depends on:
Blocks:
 
Reported: 2011-03-30 10:18 EDT by Szabolcs Bardy CLA
Modified: 2011-06-23 03:38 EDT (History)
0 users

See Also:
stepper: indigo+

Attachments
Proposed patch (1.01 KB, patch)
2011-03-30 10:21 EDT, Szabolcs Bardy CLA 		stepper: iplog+
Details | Diff
Unit test method (1.25 KB, patch)
2011-03-30 10:24 EDT, Szabolcs Bardy CLA 		stepper: iplog+
Details | Diff
Patch v2 (includes tests) (2.35 KB, patch)
2011-04-04 11:53 EDT, Eike Stepper CLA 		no flags Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Szabolcs Bardy CLA 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 CLA 2011-03-30 10:21:20 EDT 
Created attachment 192193 [details]
Proposed patch

Proposed patch for the SQLQueryHandler class.
Comment 2 Szabolcs Bardy CLA 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 CLA 2011-04-04 11:53:13 EDT 
Created attachment 192470 [details]
Patch v2 (includes tests)

Seems to work perfectly. Thanks!
Comment 4 Eike Stepper CLA 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 CLA 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 CLA 2011-04-04 21:42:29 EDT 
Thank you!

Committed revision 7586
Comment 7 Eike Stepper CLA 2011-06-23 03:38:13 EDT 
Available in R20110608-1407