Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 360018 - Deployed SQL bindings should be converted from workspace URIs to inlined
Summary: Deployed SQL bindings should be converted from workspace URIs to inlined
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Yun Feng Ma CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-05 14:17 EDT by Justin Spadea CLA
Modified: 2017-02-23 14:16 EST (History)
4 users (show)

See Also:


Attachments
Patch (12.15 KB, patch)
2011-10-12 04:53 EDT, Yun Feng Ma CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Justin Spadea CLA 2011-10-05 14:17:13 EDT
If I deploy a DD with SQL bindings using the workspace:// URI format, the resulting -bnd.xml file should have SQL bindings that have the hardcoded information. Meaning: the workspace:// URI should be used to look up the connection profile, and use it to figure out the information to place inside the -bnd.xml file.

The workspace:// URI is meaningless outside the IDE test environments, the deployed application needs the real information.
Comment 1 Brian Svihovec CLA 2011-10-10 21:09:20 EDT
Are we officially supporting the workspace:// binding  type in the .7 release, and if so, should this become a P1 enhancement, or a high severity defect?
Comment 2 Justin Spadea CLA 2011-10-11 08:46:06 EDT
I don't think this is critical to support in 0.7, just a "nice to have". If you look at REST bindings, there's nothing deployment can do with a workspace URI.
Comment 3 Yun Feng Ma CLA 2011-10-12 04:53:31 EDT
Created attachment 205005 [details]
Patch

Here is the patch. Thanks.
Comment 4 Yun Feng Ma CLA 2011-10-13 01:08:14 EDT
This fix is in, resolve this. Thanks.
Comment 5 Justin Spadea CLA 2011-11-04 14:51:45 EDT
Verified