yhknight / odata4j

Automatically exported from code.google.com/p/odata4j
0 stars 0 forks source link

Integration with Resteasy #238

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Using the Resteasy as the REST engine with OData4J does not work as some of the 
context injection used by odata4j is not supported in Resteasy. Looked at the 
JAR-311 and found that the context injection style/detail used is not defined 
by the spec thus supported by the Resteasy.

It took minor modification to odata4j classes to fix the issues to make it work 
with all the rest engines (cxf, jersy, resteasy).

The patch can be found at
https://code.google.com/r/rareddy9-teiid/source/detail?r=d1d01dc6164a3cf9fb03c09
e9e296ee7c9633ba5

I am not sure how to generate a pull request as github does, let me know if I 
need to submit that way.

Original issue reported on code.google.com by rared...@gmail.com on 7 Jan 2013 at 4:29

GoogleCodeExporter commented 9 years ago
I would also like to see this integrated into the main project.  Is this 
project dead?

Original comment by mellowaredev on 1 Apr 2013 at 4:03

GoogleCodeExporter commented 9 years ago
For the quickest path to integration, please submit this patch to our new 
gerrit instance:  review.odata4j.org

Original comment by john.spurlock on 1 Apr 2013 at 4:36

GoogleCodeExporter commented 9 years ago
Thanks John.  I am hoping the original committer will create the gerritt patch 
as this is not my code to submit!

Original comment by mellowaredev on 1 Apr 2013 at 4:55

GoogleCodeExporter commented 9 years ago
hmm, I never used gerritt, first I got find out how to use it. I wish this is 
hosted on git hub. -:(

Original comment by rared...@gmail.com on 1 Apr 2013 at 8:50

GoogleCodeExporter commented 9 years ago
Gerrit is pretty easy to set up.  Feel free to contact me directly if you have 
any questions - happy to help.

Original comment by john.spurlock on 1 Apr 2013 at 10:12

GoogleCodeExporter commented 9 years ago
Just added a "Contribution Workflow" section to our development wiki page with 
step-by-step instructions.
  https://code.google.com/p/odata4j/wiki/Development

Hope that helps,
- john

Original comment by john.spurlock on 2 Apr 2013 at 3:41

GoogleCodeExporter commented 9 years ago
Great, that helps. I need to move my changes from hg to git based repo, then 
update the Gerrit.

Original comment by rared...@gmail.com on 2 Apr 2013 at 10:58

GoogleCodeExporter commented 9 years ago
John,

It took me more than I expected, as I had to copy changed file one by one as I 
could not create a eclipse patch from one had, but here it is gerrit link

http://review.odata4j.org/5

Ramesh..

Original comment by rared...@gmail.com on 4 Apr 2013 at 8:34

GoogleCodeExporter commented 9 years ago
BTW, I have integrated OData4J into Teiid (http://teiid.org), Teiid is data 
integration engine. I basically used core odata4j engine, for its core, edm, 
language model, producer resource model. Using Teiid one can expose any data 
source as odata or consume odata source for integration with other purposes. 

https://github.com/teiid/teiid/tree/master/odata

I do think your core module has too many things in there, I would love to see 
core split into core module, and create separate jpa, jdbc, eclipselink client 
modules etc. I do have vested interest in OData4J, so, I can help however we 
can. Let me know.

Ramesh..

Original comment by rared...@gmail.com on 4 Apr 2013 at 8:38

GoogleCodeExporter commented 9 years ago
I couldn't agree more about the core having too much in it comment.  I need to 
use just a Consumer inside a Jboss 7 app and I created this defect because 
JAX-RS was colliding with CXF and had to manually comment out code to get it 
working

https://code.google.com/p/odata4j/issues/detail?id=252

Original comment by mellowaredev on 4 Apr 2013 at 8:43

GoogleCodeExporter commented 9 years ago
one more try with @john's suggestions

http://review.odata4j.org/6

Original comment by rared...@gmail.com on 5 Apr 2013 at 4:33

GoogleCodeExporter commented 9 years ago
Ramesh's changes merged into master.

Original comment by john.spurlock on 14 Apr 2013 at 4:03