Bug 1213737 - Protostream installed as EAP module does not see resources from application deployment
Summary: Protostream installed as EAP module does not see resources from application d...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Data Grid 6
Classification: JBoss
Component: Infinispan
Version: 6.5.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ER4
: 6.5.0
Assignee: Adrian Nistor
QA Contact: Martin Gencur
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-04-21 08:00 UTC by Martin Gencur
Modified: 2015-06-23 12:26 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-06-23 12:26:10 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker ISPN-5461 0 Major Resolved FileDescriptorSource.fromResources and FileDescriptorSource.addProtoFiles should have a ClassLoader argument 2015-09-11 11:40:12 UTC

Description Martin Gencur 2015-04-21 08:00:13 UTC
When I use jboss-deployment-structure to import protostream dependency to the application, the protostream library can't see classes from the application deployment. It only sees classes/resources in other modules (in its dependencies defined in module.xml).

As a result, when I want to use serCtx.registerProtoFiles(FileDescriptorSource
                .fromResources("/sample_bank_account/bank.proto")); to register protofiles on client side, I have to create a separate module with that proto file and place a dependency on it in the protostream module.

This is a usability issue, similar to https://bugzilla.redhat.com/show_bug.cgi?id=1170697 or https://bugzilla.redhat.com/show_bug.cgi?id=1206702

Comment 2 Adrian Nistor 2015-05-12 16:33:02 UTC
This needs to be fixed in protostream. I'll add a variant of the FileDescriptorSource.fromResources that accepts a ClassLoader.

But until then, this can be very easily worked around by users: use FileDescriptorSource.addProtoFile(String name, InputStream contents) instead.


Note You need to log in before you can comment on or make changes to this bug.