Re: [DISCUSS] Move microprofile impl to Apache TomEE

2022-06-14 Thread Romain Manni-Bucau
akarta) >> due to a lack of contributors / resources to ramp up updated versions >> in Geronimo. >> >> I have no strong opinion about switching the project umbrella as it >> wouldn't change anything regarding contributions, I guess. >> >> Gruß >> Richa

Re: Jakarta Mail TCK - Additional Thoughts? (was: TomEE 9.x - from javax to jakarta namespace)

2022-06-02 Thread Romain Manni-Bucau
Hi, Did you try handling LITERAL+ capability (1)? I don't think we do as of today. (1) https://datatracker.ietf.org/doc/html/rfc7888#:~:text=LITERAL%2B%20allows%20the%20alternate%20form%20of%20literals%20(called%20%22non%2D,are%204096%20bytes%20or%20less . Romain Manni-Bucau @rmannibucau <ht

Re: ValidationParser fails to parse Jakarta XML configuration files.

2022-04-06 Thread Romain Manni-Bucau
If not crazy can be great to avoid to need to rereleasz just for that else ok for me. Le mer. 6 avr. 2022 à 21:48, Jean-Louis Monteiro a écrit : > Hi, > > while working on the TomEE TCK, I realized BVal was not supporting XML > configuration because of the way we handle XML Namespaces. > > We

Re: TomEE MicroProfile and Jakarta

2022-04-01 Thread Romain Manni-Bucau
it is what it means concretely. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/

Re: TomEE MicroProfile and Jakarta

2022-03-31 Thread Romain Manni-Bucau
ave to go that way at some point anyway, ideas welcomed. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://w

Re: JSON Pointer/Patch issue with /-

2020-12-02 Thread Romain Manni-Bucau
Let say it enables more control whereas this kind of toggle system properties can be random when apps don't need the same (once again it is functional and not about perf there). Feel free to take back my PR, was just sharing the idea with code to try to fix it together faster. Romain Manni-Bucau

Re: JSON Pointer/Patch issue with /-

2020-12-02 Thread Romain Manni-Bucau
Send a PR with the "SPI" option which enables to have this toggle *at will* and drop it when not desired anymore without any config. Hope it illustrates better than words one toggle option which wouldnt depend on the env. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannib

Re: JSON Pointer/Patch issue with /-

2020-12-01 Thread Romain Manni-Bucau
createPointerFactory(mapWithToggle) so it is a global flag which means it breaks some deployments anyway - at least at tomee level - when > 1 app is deployed (or >= 1 app + 1 extension). > > On Fri, Nov 27, 2020 at 10:44 AM Romain Manni-Bucau > > wrote: > > > Hi JL, > &

Re: JSON Pointer/Patch issue with /-

2020-11-27 Thread Romain Manni-Bucau
r everyone for literally 0 gain IMHO. Hope it makes sense. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | Link

Re: WebApplicationExceptionMapper not being called

2019-12-25 Thread Romain Manni-Bucau
Hi Ivan You can set it with a cxf interceptor or even jaxrs filter but looks like a bug, at least the default. Default should be true only if there is no custom mapper, not if there is any registered mapper matching and this is known at deploy time. Guess you should open a ticket. Le mer. 25

Re: Implementing MicroProfile 3.0

2019-09-23 Thread Romain Manni-Bucau
issues to version Health_1.0.3 > <https://issues.apache.org/jira/browse/GERONIMO/fixforversion/12346154> > > > > > -- > Jean-Louis Monteiro > http://twitter.com/jlouismonteiro > http://www.tomitribe.com > > > On Sun, Sep 22, 2019 at 11:00 PM Romain Manni-Bucau >

Re: Implementing MicroProfile 3.0

2019-09-23 Thread Romain Manni-Bucau
Hi JL AFAIK the missing impl are mainly fault tolerance 2 (which adds completionstage support we already have), opentracing 1.3 (which is sadly just minor fixes compared to others and no opentracing upgrade IIRC), and metrics 2 (which mainly split counters in 2 entities). Other specs are ready I

Re: Johnzon release

2019-09-11 Thread Romain Manni-Bucau
TCK small so we do a 1.1.14 or > do we consider they are significant enough to produce a 1.2 instead? > > thoughts? > -- > Jean-Louis Monteiro > http://twitter.com/jlouismonteiro > http://www.tomitribe.com > > > On Mon, Sep 9, 2019 at 12:05 PM Romain Manni-Bucau > wro

Re: Johnzon release

2019-09-09 Thread Romain Manni-Bucau
As mentionned in the related thread ("JSON-B and JSON-P TCKs") it sounds ok for me. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://git

Re: DISCUSS geronimo-security_1.0_spec content unclear

2019-09-04 Thread Romain Manni-Bucau
We cant, only impl+api are certified so no issue ;) Le mer. 4 sept. 2019 à 17:01, Jean-Louis Monteiro a écrit : > I'd like to certify some of them if possible of course. > > Le mer. 4 sept. 2019 à 15:33, Romain Manni-Bucau > a écrit : > >> Not sure I'm following Mar

Re: DISCUSS geronimo-security_1.0_spec content unclear

2019-09-04 Thread Romain Manni-Bucau
Not sure I'm following Mark, EPL is fine for us ( https://www.apache.org/legal/resolved.html) and G spec jars are not officially certified so don't change of license anytime. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/>

Re: DISCUSS geronimo-security_1.0_spec content unclear

2019-09-04 Thread Romain Manni-Bucau
@Mark: didn't change with jakarta donation? can you open a ticket on jakartee tracker please? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://g

Re: DISCUSS geronimo-security_1.0_spec content unclear

2019-09-04 Thread Romain Manni-Bucau
if system props/SPI enable to switch it since it enables to make fatjars smooths without configs and to ensure we load the right default but it is really a single string so can be discussed. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.met

Re: DISCUSS geronimo-security_1.0_spec content unclear

2019-09-04 Thread Romain Manni-Bucau
No I guess it was right, "that are" ;) = fork @G only when we need to change some impl/default provider. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com>

Re: DISCUSS geronimo-security_1.0_spec content unclear

2019-09-03 Thread Romain Manni-Bucau
go ahead Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> |

Re: DISCUSS geronimo-security_1.0_spec content unclear

2019-09-03 Thread Romain Manni-Bucau
MP license is ok (Apache2) but Jakarta is EPLs so keeps the ambiguity for us. That said it is good to reuse the same GAV for end users so we might ask jakarta to double license its api jars? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Re: DISCUSS geronimo-security_1.0_spec content unclear

2019-09-03 Thread Romain Manni-Bucau
gt; depends what their license is. EPL is (weak) copyleft. Thus I would like >> to avoid exposing it downstream as api. >> >> LieGrue, >> strub >> >> >> > Am 03.09.2019 um 16:20 schrieb Romain Manni-Bucau < >> rmannibu...@gmail.com>: >>

Re: DISCUSS geronimo-security_1.0_spec content unclear

2019-09-03 Thread Romain Manni-Bucau
If we still can't reuse jakata artifacts (their license is ok and there is no impl reference inside so we should just use them, right?) it sounds natural Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Ol

Re: JSON-B and JSON-P TCKs

2019-08-26 Thread Romain Manni-Bucau
was planned, we were waiting Mark green light IIRC Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn &

Re: JPA Spec jar missing Activator

2019-08-14 Thread Romain Manni-Bucau
Hi JL, Not sure if you need it - which can make us reconsider it - but long story short it is superseeded by capabilities on modern OSGi containers so all done in the pom. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/>

Wrong m3 bval dep?

2019-08-02 Thread Romain Manni-Bucau
Hi everybody Last m3 depends on a phantom bval release breaking dependency management it seems. Was it identified/communicated?

Re: OpenTracing Zipkin

2019-04-15 Thread Romain Manni-Bucau
wasn't able to make it work. > > I will have a look at it again this week, if I have improvement I will let > you know. > > If you have any other tips let me know. > > Thanks for the reply! > > On Fri, Apr 12, 2019 at 6:25 PM Romain Manni-Bucau > wrote: > &g

Re: OpenTracing Zipkin

2019-04-12 Thread Romain Manni-Bucau
Hi Ivan Did you debug ZipkinHttp? Depending your server/app config you can need some config like providers etc. This class is a good start to check what is happening. A good config start is https://github.com/jeanouii/microprofile-samples/blob/master/docker-compose.yaml - you can need to inline

Re: RestClient issue - different apps with same client

2019-03-12 Thread Romain Manni-Bucau
Cxf rest cdi extension leaks beans, think i opened n issue last week or the week before about it. Dropping the static from the extension fixes that. Guess it was done as a workaround for ears but probably better to use another mecanism like the one of deltaspike or just not do it. Le mar. 12

Re: "In" parameter not being populated OpenAPI

2019-02-11 Thread Romain Manni-Bucau
gth": 2147483647, > "minLength": 0, > "nullable": false, > "readOnly": false, > "type": "string", > "uniqueItems": false, >

Re: "In" parameter not being populated OpenAPI

2019-02-11 Thread Romain Manni-Bucau
. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book <htt

Re: "In" parameter not being populated OpenAPI

2019-02-11 Thread Romain Manni-Bucau
Yes Ivan, array mapping is in progress. In the meantime you can define your schema to ensure you control it and the implicit representation does not depends on the way the impl parses it - which can not match your underlying mapper. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibu

Re: "In" parameter not being populated OpenAPI

2019-02-11 Thread Romain Manni-Bucau
Hi Ivan, Did you test on the snapshot? we got some enhancements about it. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibuc

Re: @OpenAPIDefinition not working

2019-02-08 Thread Romain Manni-Bucau
Hey, just recalled we had a flag about it, you can skip it setting openejb.cxf-rs.cache-application=false Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github &

Re: @OpenAPIDefinition not working

2019-02-08 Thread Romain Manni-Bucau
. Enhancing TomEE to no do it anymore or not use a wrapper when not needed can be a first step fixing that. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github &

Re: OpenTracing - nullpointer OpenTracingFilter

2019-01-22 Thread Romain Manni-Bucau
for instance) and forcing geronimo.opentracing.filter.active to false for the webapp (not globally). Setting the system property you should get the same behavior but it will be global so some app will not get tracing. In other word it is the core work tomee must do: integration :). Romain Manni-Bucau

Re: Adding Java EE Security 1.0 Spec to geronimo-specs

2019-01-11 Thread Romain Manni-Bucau
If Roberto agrees +1 Le ven. 11 janv. 2019 20:29, Gurkan Erdogdu a écrit : > Hi folks > Regarding TomEE, we need to add Java EE Security, JSR-375 spec to > geronimo-specs and needs a release. I have committer access to > geronimo-specs and added to code (implemented by Roberto Cortez in >

Re: TOMEE-2289

2019-01-04 Thread Romain Manni-Bucau
that yaml is supposed to be the default then we are in a grey enough area to make you happy ;). Worse case we would revert it but it is not a prod usage for sure so no big deal for now. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.met

Re: New Java EE Schemas for Java EE Deployment Descriptors

2018-12-02 Thread Romain Manni-Bucau
Hello If it helps there are twi things to consider: 1. When we supported ee7 we made most of these descriptors at lowest cost which means we mainly let them pass and sometimes ignore some model part not used at runtime and let the actual runtime lib read it (typically jsf) 2. All the current

Re: Merging Old and New Websites

2018-12-02 Thread Romain Manni-Bucau
If it helps I have an antora setup ([1]) to generate another doc site with most of these features - think only the day/night thing is missing, it uses tags to manage versions - overridable by branches. Side note: when we did the new site we decided to not merge with the old one cause too much

Re: Microprofile OpenAPI

2018-11-30 Thread Romain Manni-Bucau
ill open about this issue that can > update or not later the fix in the TCK. > > > > [1] https://github.com/eclipse/microprofile-open-api/issues/228 > [2] https://github.com/eclipse/microprofile-open-api/issues/302 > > > El vie., 30 nov. 2018 a las 10:49, Romain Manni-Bu

Re: Microprofile OpenAPI

2018-11-30 Thread Romain Manni-Bucau
current impl not compliant? > > Like I said yaml is not default and that diverges from the spec. Adding > yaml as default if we add jackson yaml won't make it default as well, it > will be still an alternative. > Ok, then this is this assumption which is wrong and where this thread beca

Re: Microprofile OpenAPI

2018-11-30 Thread Romain Manni-Bucau
Ok, will repeat a third time hoping it is a mail crossing issue: why is current impl not compliant? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://g

Re: Microprofile OpenAPI

2018-11-30 Thread Romain Manni-Bucau
for "other$" reasons. Also note that, as I mentionned, the implementation is now compliant to the spec, so not really sure the follow up to give to that topic. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.me

Re: Microprofile OpenAPI

2018-11-30 Thread Romain Manni-Bucau
to be compatible, this is what has been done in TomEE since years and it works well making users happy rather than spec leads. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com&g

Re: Microprofile OpenAPI

2018-11-30 Thread Romain Manni-Bucau
If jackson yaml is present it will add a (jackson) writer for yaml, if not it stays on json. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://g

Re: Forking for new PRs

2018-11-30 Thread Romain Manni-Bucau
'm in this last partbeing said at the end you can't rely on the history whatever effort you put in the writing of it). I have a colleague who is quite interesting on that, he says "I squash 'fix typo' commits only". Romain Manni-Bucau @rmannibucau <https://twitte

Re: Microprofile OpenAPI

2018-11-30 Thread Romain Manni-Bucau
I'll add */* for yaml endpoint to complement text/plain, this will solve the ambiguity Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rm

Re: Microprofile OpenAPI

2018-11-29 Thread Romain Manni-Bucau
mo/microprofile/openapi/jaxrs/OpenAPIFilter.java#L57 > > On Thu, Nov 29, 2018 at 3:58 PM Romain Manni-Bucau > wrote: > >> Response is fine (thanks jaxrs), request is up to jaxrs runtime so >> depends where you deploy it (i dont think implementing a custom writer for >> that is

Re: Microprofile OpenAPI

2018-11-29 Thread Romain Manni-Bucau
Response is fine (thanks jaxrs), request is up to jaxrs runtime so depends where you deploy it (i dont think implementing a custom writer for that is right for users, it has too much pitfalls once integrated to anything else than this very specific spec). Le jeu. 29 nov. 2018 21:39, Jonathan

Re: Microprofile OpenAPI

2018-11-29 Thread Romain Manni-Bucau
Nov 29, 2018 at 11:03 AM Ivan Junckes Filho > > wrote: > > > Hey Romain, take a look on: > > > > > > > https://download.eclipse.org/microprofile/microprofile-open-api-1.0/microprofile-openapi-spec.html#_content_format > > > > It is clear for me there,

Re: Microprofile OpenAPI

2018-11-29 Thread Romain Manni-Bucau
ver be) so no reason to import a lib (which can be heavy and potentially with vulnerabilities + work for the users to maintain it) for that. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordp

Re: [TOMEE-2287] Histogram Prometheus Output

2018-11-28 Thread Romain Manni-Bucau
it on your side, did you set the system property geronimo.metrics.jaxrs.activated=true ? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rm

Re: Please review PR #214

2018-11-26 Thread Romain Manni-Bucau
Hello Bruno, I put some suggestions on the PR, hope it helps. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn &

Re: Please review PR #201

2018-11-26 Thread Romain Manni-Bucau
@Bruno: note that this is not what we are doing, I'm just mentionning that TomEE does not need that and that there is no need to put any pressure either on TomEE or Geronimo in such situation since everything is good on both side in current state. Romain Manni-Bucau @rmannibucau <ht

Re: Avoid String in a loop

2018-11-26 Thread Romain Manni-Bucau
+0 (this is only for error cases as far as i saw and a few concatenations so you don't see the diff in practise + the compiler is able to optimize it and even replace string builder when relevant these days) Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Re: Please review PR #201

2018-11-26 Thread Romain Manni-Bucau
ers. > > Bruno Baptista > https://twitter.com/brunobat_ > > > On 23/11/18 15:41, Romain Manni-Bucau wrote: > > Le ven. 23 nov. 2018 à 16:34, Bruno Baptista a > écrit : > > > >> Hi Romain, > >> > >> About "The point is not the cdi bean but t

Re: Building TomEE, openejb-core, crashed test: org.apache.openejb.assembler.DeployerEjbTest

2018-11-25 Thread Romain Manni-Bucau
Hi Ferdi, surefire got some "hiccup" in last versions, maybe try 3.0.0-M1 or or 2.22.1 which should work, if not the output dump can help sometimes to identify if it is a memory issue or (more likely) another one. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau>

Re: Feedback as newbie

2018-11-24 Thread Romain Manni-Bucau
/community/index.adoc). And yes we never used develop branch, only master. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibuc

Re: Feedback as newbie

2018-11-24 Thread Romain Manni-Bucau
Hello Quick recommandation for such a very valuable feedback is to make it a contribution doing a PR on http://github.com/apache/tomee-site-generator and let tomee community promote it live asap. Also it encourages it to be part of the community work quickly and avoid it to be lost. This way you

Re: Write tests geronimo

2018-11-23 Thread Romain Manni-Bucau
Hi Ivan TCK are in geronimo and they use arquillian, tomee just reapplies them to check the packaging does not break anything and its classloading is fine as well so you can write tests the same. Le ven. 23 nov. 2018 19:51, Ivan Junckes Filho a écrit : > Hello guys, I did a change on

Re: Metrics Gauge Example and Bug

2018-11-23 Thread Romain Manni-Bucau
https://github.com/apache/tomee/blob/4c7fd4af95983a92bef89dc598873310dd13dd2e/server/openejb-cxf-rs/src/test/java/org/apache/openejb/server/cxf/rs/johnzon/JsonbJaxrsProviderTest.java was supposed to cover that Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Re: Please review PR #201

2018-11-23 Thread Romain Manni-Bucau
the default executor as being lazy 3. we assume safeguard will not impact an app not using it ==> you must ensure that 3 didnt trigger an executor creation, it is fine to rely on 1+2 (which means so "main" code) > > Cheers > > Bruno Baptista > https://twitter.com/brunobat_ >

Re: Please review PR #201

2018-11-23 Thread Romain Manni-Bucau
cify he wants safeguard to use the executor "'mysafeguardexecutor". This is why the config is important and I mentionned it early even if it is not the most sexy part to do, I agree. > > Cheers > > Bruno Baptista > https://twitter.com/brunobat_ > > > On 23/1

Re: Please review PR #201

2018-11-23 Thread Romain Manni-Bucau
hich is quite ridiculous when you think about it for something about failt tolerance. This is why it should be in before next release. Now if you do the PR next week it is fine, was not to do it today but to ensure it is not merged and the enthusiasm makes it forgotten. Romain Manni-Bucau @rmann

Re: MicroProfile rest client example

2018-11-22 Thread Romain Manni-Bucau
Hi César Not sure the snapshot was deployed, maybe build it locally JL's fix does not change that error which is an outdated api in the server. Le ven. 23 nov. 2018 03:58, César Hernández Mendoza a écrit : > Hi, > I updated my PR [1], the previous exception is no longer appearing since > I'm

Re: Metrics Gauge Example and Bug

2018-11-22 Thread Romain Manni-Bucau
eated the PR but the >>> microprofile metrics TCK seems to be broken on TomEE, so I am not sure if >>> the PR is reliable. >>> https://github.com/apache/geronimo-metrics/pull/2 >>> >>> I will try to check what is going on with the TCK on TomEE,

Re: TOMEE-2253 - tomee.sh -version not working properly with Java 11

2018-11-22 Thread Romain Manni-Bucau
modify the classloader by default at the beginning of the review and just reuse the system classpath. The intention was to make this custom loader behave the same as the child first one but guess it was never aligned. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Re: Metrics Gauge Example and Bug

2018-11-21 Thread Romain Manni-Bucau
Hi Ivan It is a bug in tomee scanning I think Le mer. 21 nov. 2018 21:35, Ivan Junckes Filho a écrit : > Hey guys, I was writing an example of metrics gauge (WIP). > > https://github.com/apache/tomee/pull/213 > > And I found a bug when trying access a gauge with "Accept: > application/json".

Re: Is there someone looking for Microprofile Reactive at TomEE?

2018-11-21 Thread Romain Manni-Bucau
dency free since the spec does nothing much compared to the jvm itself. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | Lin

Re: Is there someone looking for Microprofile Reactive at TomEE?

2018-11-20 Thread Romain Manni-Bucau
Ok so MP decided to be bloated as the old ee so we will get a light impl @geronimo - likely next year. Anyone wants to lead it since you asked? Le mar. 20 nov. 2018 13:44, Romain Manni-Bucau a écrit : > Hi Otavio, > > none yet and the existing ones are either bloated or don't integ

Re: Microprofile Metrics Example

2018-11-20 Thread Romain Manni-Bucau
been fixed for microprofile metrics 2.x > release [1] [2] as we are speaking. > > [1] https://github.com/eclipse/microprofile-metrics/pull/309 > [2] https://github.com/eclipse/microprofile-metrics/issues/290 > > > El lun., 19 nov. 2018 a las 23:52, Romain Manni-Bucau (< > r

Re: TOMEE-2253 - tomee.sh -version not working properly with Java 11

2018-11-20 Thread Romain Manni-Bucau
..@gmail.com> escreveu: > >> > > >> > > I'm following this thread with some interest. What would be some > good > >> > tests > >> > > to make sure these cases are covered off? Some suggestions in > >> addition to > >> > > the existing test: > >

Re: About TomEE Twitter account

2018-11-20 Thread Romain Manni-Bucau
What about a bot polling tweets in a github repo - with a scheduled date for instance? Then PR would be welcomed ;). Just needs a server (or even a free heroku pby or openshift). Le mar. 20 nov. 2018 18:57, David Blevins a écrit : > I would trust Mark Struberg with access to the twitter account

Re: Is there someone looking for Microprofile Reactive at TomEE?

2018-11-20 Thread Romain Manni-Bucau
Hi Otavio, none yet and the existing ones are either bloated or don't integrate well in any existing stack :( FYI the reason it is not @geronimo: https://github.com/eclipse/microprofile/issues/57 Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Re: TomEE 8 - Next steps

2018-11-19 Thread Romain Manni-Bucau
Probably worth a read: http://tomee-openejb.979440.n4.nabble.com/VOTE-next-tomee-version-targetting-EE-7-td4674650.html http://tomee-openejb.979440.n4.nabble.com/Java-EE-version-Alignment-Troubles-td4674796.html And related if you still feel motivated by reading after these pages ;) Le mar. 20

Re: Microprofile Metrics Example

2018-11-19 Thread Romain Manni-Bucau
Hi César, Did you test passing accept header or just in the browser which sends json and text as acceptable value so resolution can be not that deterministic using it - but this is the request so all fine ;). The last side note is prometheus does not support monotonic=false (small glitch in the

Re: Override bean from library on TomEE

2018-11-19 Thread Romain Manni-Bucau
tomee/commit/23f44b17e6ab504561f32aa4ae92f2521e07261e > However, the problem is before that... The test with the interceptor, > somehow fails to load the alternative, which has me very puzzled. > > Cheers. > > Bruno Baptista > http://twitter.com/brunobat_ > > > On 19/11/18 16:45, Romain Man

Re: Override bean from library on TomEE

2018-11-19 Thread Romain Manni-Bucau
n as the injection mystery > is solved. Any feedback on the injection of the alternative bean will be > appreciated because I'm kind of stuck now. > > > > Cheers > > > > Bruno Baptista > > http://twitter.com/brunobat_ > > > > > > On 19/11/18 15

Re: Override bean from library on TomEE

2018-11-19 Thread Romain Manni-Bucau
ode this bean themselves to control it I think. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://w

Re: Override bean from library on TomEE

2018-11-16 Thread Romain Manni-Bucau
() { @Override // hardcoded impl for testing purposes public Object execute(final InvocationContext invocationContext) { return "replaced"; } }; } } Side note: did you want to use @Resource for the executor injection? Romain Manni-Bucau @rmannibu

Re: Override bean from library on TomEE

2018-11-16 Thread Romain Manni-Bucau
Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book <htt

Re: Override bean from library on TomEE

2018-11-16 Thread Romain Manni-Bucau
Hi Bruno, it is a palin bean so @Specializes works, did you put it on the method? Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rm

Re: tomee git commit: TOMEE-2277 Java 11 - define modules for TomEE At the moment, they are tight to maven artifact but we can improve later.

2018-11-15 Thread Romain Manni-Bucau
The full tomee build does not rely on modules even ran with java11 so it does not surprises me much. Try to create a new project and require openejb-core module for instance. Will be enough to check Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog

Fwd: tomee git commit: TOMEE-2277 Java 11 - define modules for TomEE At the moment, they are tight to maven artifact but we can improve later.

2018-11-15 Thread Romain Manni-Bucau
Hey JL, did you check these names are valid? Think the - can be an issue (jdk.internal.module.Checks#requireModuleName) Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com&g

Fwd: tomee git commit: TomEE-2274 Java 11 compatibility - upgrade to Jaxb 2.3.1 Thanks Cees Bos

2018-11-13 Thread Romain Manni-Bucau
Hi JL, did you mean 2.3.0.1? 2.3.1 does not exist under these coordinates also take care it is built with java 9 so maybe not what we want to deliver in tomee Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Ol

Re: Some jars in TomEE 8 SNAPSHOT are not Java 11 compatible

2018-11-13 Thread Romain Manni-Bucau
I'm running cxf under java 11 since some time now so it can be important in that work to split the statements between "it runs on java11" and "it uses java11 natively" - which is not done by any lib today AFAIK. Romain Manni-Bucau @rmannibucau <https://twitter.com/rman

Re: HikariCP Connection Pooling

2018-11-12 Thread Romain Manni-Bucau
Side note: tomee already supports it - or any fancy pool - through resource definition Le lun. 12 nov. 2018 22:29, Jonathan Gallimore a écrit : > Yeah, I saw that a while back and thought it was interesting. I'd be > interested in adding support for it in TomEE. > > Jon > > On Mon, Nov 12, 2018

Re: Java 11 status

2018-11-11 Thread Romain Manni-Bucau
- if needed). Overall point was to not lock an ecosystem for features not used in half of it. Commons-[pool|dbcp] is not required dependency of openjpa so shouldnt block its release and if it does we must work to fix that issue IMHO. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibu

Re: Java 11 status

2018-11-10 Thread Romain Manni-Bucau
f OpenJPA hacking. > One of the tasks is to also support xbean-asm7. > That should go into openjpa-3.0.1. > > What is the timeframe we want to target? > > LieGrue, > strub > > > > Am 09.11.2018 um 14:18 schrieb Romain Manni-Bucau >: > > > > I don't h

Re: Java 11 status

2018-11-09 Thread Romain Manni-Bucau
I don't have the list handy but tomee will need at least some --add-opens etc to run on java11 (you can add it in setenv.sh) - likely for unsafe and the agent. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Ol

Re: Java 11 status

2018-11-08 Thread Romain Manni-Bucau
Hi JL Openjpa needs xbean upgrade but should be quick. Cxf stack needs some jaxb upgrades from memory. After the stack "supports" java11 as "run it" but think no spec in EE is java >= 9 compatible (modules, spi etc). Should be enough for tomee that said. Le jeu. 8 nov. 2018 20:27, Jean-Louis

Re: TomEE 8 M1 - MP

2018-11-05 Thread Romain Manni-Bucau
The built-in apps should have a MP config entry with the value geronimo.opentracing.filter.active=false, you can workaround it adding in the exploded webapp a WEB-INF/classes/META-INF/microprofile-config.properties with this value Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibu

Re: MicroProfile JWT 1.1

2018-11-02 Thread Romain Manni-Bucau
s efforts, but this has probably contributed to some confusion. I > would suggest we continue this over on the Geronimo and OWB lists to avoid > further confusion. > > Jon > > On Fri, Nov 2, 2018 at 12:46 PM Romain Manni-Bucau > wrote: > >> Hi >> >> Yes t

Re: MicroProfile JWT 1.1

2018-11-02 Thread Romain Manni-Bucau
erBasedProducer class, and > that looks like a security issue. > > Jon > > On Tue, Oct 30, 2018 at 5:56 AM Romain Manni-Bucau > wrote: > > > Hi Jon, > > > > yes and no, idea is to be fast and for all producers it works except the > > principal which i

Re: Finish the bval setup

2018-10-31 Thread Romain Manni-Bucau
It is mainly about ensuring it is ok then merge yes The jfx tests are not executed if not configured as such, the flag does it If I got it right Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Ol

Re: TOMEE-2253 - tomee.sh -version not working properly with Java 11

2018-10-31 Thread Romain Manni-Bucau
merge this in with confidence, and without the tests I > don't > > see how we can do that. > > > > Jon > > > > On Mon, Oct 29, 2018 at 3:35 PM Romain Manni-Bucau < > rmannibu...@gmail.com> > > wrote: > > > > > Hi Daniel, > > > > &

Finish the bval setup

2018-10-31 Thread Romain Manni-Bucau
PR and potentially do the javafx setup it would be awesome. Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedI

Re: MicroProfile JWT 1.1

2018-10-29 Thread Romain Manni-Bucau
/microprofile/impl/jwtauth/tck/TckSecurityService.java Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.

Re: Shutdown issues with ManagedScheduledExecutorService

2018-10-29 Thread Romain Manni-Bucau
Did you try a semaphore waiting all tasks are done? It likely also needs a boolean to prevent new submissions before starting waiting for completion. Side note: @Schedule sounds not bad for your use case ;) Le lun. 29 oct. 2018 20:55, ChrisOwens a écrit : > This is with TomEE plus 7.0.5 > >

  1   2   3   4   5   6   7   8   9   10   >