You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2020/06/22 12:48:23 UTC
[GitHub] [camel] davsclaus opened a new pull request #3939: Camel 15224
davsclaus opened a new pull request #3939:
URL: https://github.com/apache/camel/pull/3939
Just a heads up about a big commit coming to make the api components reflection free. This is done by generating configurers for their generated configuration classes (double generation). And now also store this output in src/generated so its more visible to us when there are changes.
There is one place where reflection is still in use as there is a diff between refelection vs source code generated (I have a TODO). If that gets resolved then there is no reflection anymore. Currently one of the olingo4 tests shows that the bean introspector is used 14 times, it should be 0 for reflection free.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [camel] davsclaus closed pull request #3939: Camel 15224
Posted by GitBox <gi...@apache.org>.
davsclaus closed pull request #3939:
URL: https://github.com/apache/camel/pull/3939
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [camel] davsclaus commented on pull request #3939: Camel 15224
Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #3939:
URL: https://github.com/apache/camel/pull/3939#issuecomment-647610413
Argh fuck a git push origin also mistakenly pushed some changes on my master which was branched for this. I wanted to do a squash when this PR was ready. Okay sorry, so I will have to merge the 2 new commits back on master and close this PR
----------------------------------------------------------------
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:
users@infra.apache.org