[GitHub] [camel] onderson edited a comment on pull request #3777: CAMEL-2933 - resteasy component

2020-05-10 Thread GitBox


onderson edited a comment on pull request #3777:
URL: https://github.com/apache/camel/pull/3777#issuecomment-626411872


   subtasks are created, considering enhancements.
   https://issues.apache.org/jira/browse/CAMEL-15038
   https://issues.apache.org/jira/browse/CAMEL-15039
   https://issues.apache.org/jira/browse/CAMEL-15040
   https://issues.apache.org/jira/browse/CAMEL-15041
   https://issues.apache.org/jira/browse/CAMEL-15042



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [camel] onderson edited a comment on pull request #3777: CAMEL-2933 - resteasy component

2020-05-06 Thread GitBox


onderson edited a comment on pull request #3777:
URL: https://github.com/apache/camel/pull/3777#issuecomment-624672808


   @davsclaus, i took a look at some part of it. not sure about what to do with 
the proxy thing. it is not super clear to either, this might be what we can do 
with resteasy till they will get rid of reflections and some part of it.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [camel] onderson edited a comment on pull request #3777: CAMEL-2933 - resteasy component

2020-05-01 Thread GitBox


onderson edited a comment on pull request #3777:
URL: https://github.com/apache/camel/pull/3777#issuecomment-62237


   > Its a massive work to write a HTTP component with both consumer/producer 
and rest-dsl support and whatnot. Great work so far, it must have taken a lot 
of time.
   > 
   > Its a bit hard to review when there is so much code. So lets get the proxy 
thing figured out, if its really something you need. Would be kinda bad if you 
must write a interface class just to use this component. But its not clear to 
me.
   
   thanks @davsclaus, I should say most of the credits should go to @rjakubco, 
i made a rework on his very old PR with newer versions of resteasy and camel. 
reviews are definetely helpful to enhance and remove parts we would not need. i 
will try to readdress and if there is any more reviews, more than welcome.
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org