[GitHub] [tomee] dblevins commented on issue #422: Creates Unit test to CmpJpaConversion

[GitBox]

dblevins commented on issue #422: Creates Unit test to CmpJpaConversion
URL: https://github.com/apache/tomee/pull/422#issuecomment-472235859
 
 
   Can you also do the second part, which is update the community via the dev 
list on what problem this PR solves and what people should know so they can 
review 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


With regards,
Apache Git Services

Re: RestClient issue - different apps with same client

[j4fm]

https://issues.apache.org/jira/browse/CXF-7980




--
Sent from: http://tomee-openejb.979440.n4.nabble.com/TomEE-Dev-f982480.html

[GitHub] [tomee] ivanjunckes commented on issue #438: CXF 3.3.1 + MP Rest Client 1.2.1

[GitBox]

ivanjunckes commented on issue #438: CXF 3.3.1 + MP Rest Client 1.2.1
URL: https://github.com/apache/tomee/pull/438#issuecomment-472174058
 
 
   @jgallimore @jeanouii can you guys merge?


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


With regards,
Apache Git Services

[GitHub] [tomee] j4fm commented on issue #438: CXF 3.3.1 + MP Rest Client 1.2.1

[GitBox]

j4fm commented on issue #438: CXF 3.3.1 + MP Rest Client 1.2.1
URL: https://github.com/apache/tomee/pull/438#issuecomment-472173882
 
 
   Hi @ivanjunckes, it's ready to be merged.
   
   Thanks


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


With regards,
Apache Git Services

Re: RestClient issue - different apps with same client

[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 mars 2019 à 20:46, Daniel Cunha  a écrit :

> Hi Ivan,
>
> Can you share a sample?
>
> Em ter, 12 de mar de 2019 às 16:34, Ivan Junckes Filho <
> ivanjunc...@gmail.com> escreveu:
>
> > Hey guys, there is a very annoying bug when using rest client from cxf
> > 3.2.7 in TomEE M2. When 2 apps have the same client the server throws the
> > error below and shuts down?
> >
> > Shouldn't we allow different apps in the same server use the same client?
> >
> > *Error*
> > Caused by: org.apache.webbeans.exception.WebBeansException:
> > org.apache.webbeans.exception.DuplicateDefinitionException:
> > PassivationCapable bean id is not unique:
> > br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient
> > bean:SiemClassificacaoServiceClient, WebBeansType:THIRDPARTY,
> >
> >
> Name:br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient,
> > API
> >
> >
> Types:[br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient],
> >
> >
> Qualifiers:[javax.enterprise.inject.Default,org.eclipse.microprofile.rest.client.inject.RestClient,javax.enterprise.inject.Any],
> > existing: SiemClassificacaoServiceClient, WebBeansType:THIRDPARTY,
> >
> >
> Name:br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient,
> > API
> >
> >
> Types:[br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient],
> >
> >
> Qualifiers:[javax.enterprise.inject.Default,org.eclipse.microprofile.rest.client.inject.RestClient,javax.enterprise.inject.Any]
> > at
> >
> >
> org.apache.webbeans.event.ObserverMethodImpl.notify(ObserverMethodImpl.java:371)
> > at
> >
> >
> org.apache.webbeans.event.NotificationManager.invokeObserverMethod(NotificationManager.java:818)
> > at
> >
> >
> org.apache.webbeans.event.NotificationManager.fireEvent(NotificationManager.java:714)
> > ... 42 more
> >
>
>
> --
> Daniel "soro" Cunha
> https://twitter.com/dvlc_
>

Trying to summarize our process

[Richard Monson-Haefel]

Hi,

I'm writing a blog post which will attempt to summarize our process for
using JIRA with the TomEE project.  Below is a link to a diagram I drew up.
It would be great if to get feedback from both veterans and people new to
the community to see if this jives with your understanding of the process.

Diagram Link
https://www.dropbox.com/s/58imhmpoh6bgfok/Image%20X%20-%20Process%20Image.png?dl=0

Thank you!

Richard

-- 
Richard Monson-Haefel
https://twitter.com/rmonson
https://www.linkedin.com/in/monsonhaefel/

Re: RestClient issue - different apps with same client

[Daniel Cunha]

Hi Ivan,

Can you share a sample?

Em ter, 12 de mar de 2019 às 16:34, Ivan Junckes Filho <
ivanjunc...@gmail.com> escreveu:

> Hey guys, there is a very annoying bug when using rest client from cxf
> 3.2.7 in TomEE M2. When 2 apps have the same client the server throws the
> error below and shuts down?
>
> Shouldn't we allow different apps in the same server use the same client?
>
> *Error*
> Caused by: org.apache.webbeans.exception.WebBeansException:
> org.apache.webbeans.exception.DuplicateDefinitionException:
> PassivationCapable bean id is not unique:
> br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient
> bean:SiemClassificacaoServiceClient, WebBeansType:THIRDPARTY,
>
> Name:br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient,
> API
>
> Types:[br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient],
>
> Qualifiers:[javax.enterprise.inject.Default,org.eclipse.microprofile.rest.client.inject.RestClient,javax.enterprise.inject.Any],
> existing: SiemClassificacaoServiceClient, WebBeansType:THIRDPARTY,
>
> Name:br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient,
> API
>
> Types:[br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient],
>
> Qualifiers:[javax.enterprise.inject.Default,org.eclipse.microprofile.rest.client.inject.RestClient,javax.enterprise.inject.Any]
> at
>
> org.apache.webbeans.event.ObserverMethodImpl.notify(ObserverMethodImpl.java:371)
> at
>
> org.apache.webbeans.event.NotificationManager.invokeObserverMethod(NotificationManager.java:818)
> at
>
> org.apache.webbeans.event.NotificationManager.fireEvent(NotificationManager.java:714)
> ... 42 more
>


-- 
Daniel "soro" Cunha
https://twitter.com/dvlc_

Re: Issue with @JohnzonConverter

[Daniel Cunha]

Hi Ivan,

for sure you need to know the data format that you are expecting, by
default Apache Johnzon uses the pattern “MMddHHmmssZ”
You can use the JAXB converter to do it, as Romain describe here in his
blog:
https://rmannibucau.wordpress.com/2015/07/09/javaee-apache-johnzon-json-library-javascript-date/

Em ter, 12 de mar de 2019 às 15:18, Ivan Junckes Filho <
ivanjunc...@gmail.com> escreveu:

> I know Dani, that is the issue. But how to fix it? It should have been
> converting to the right format.
>
> On Tue, Mar 12, 2019 at 2:51 PM Daniel Cunha 
> wrote:
>
> > Well.. seems that you convert expect that: dd/MM/ HH:mm:ss
> > (
> >
> >
> https://github.com/ivanjunckes/convertertest/blob/master/src/main/java/com/test/DateTimeConverter.java#L17
> > )
> > But you are receiving that: 2019-03-12T17:14:31.941
> >
> > Em ter, 12 de mar de 2019 às 14:19, Ivan Junckes Filho <
> > ivanjunc...@gmail.com> escreveu:
> >
> > > Hey guys, can anyone help me to understand why I am having issues with
> > the
> > > DateTimeConverter in my test?
> > >
> > > Here is a sample project with the issue.
> > > https://github.com/ivanjunckes/convertertest
> > >
> > > If anyone could also help make this project work without the need for
> the
> > > DateTimeConverter but actually using a configuration properties would
> be
> > > great.
> > >
> > > *Error:*
> > > Caused by: java.lang.IllegalArgumentException:
> java.text.ParseException:
> > > Unparseable date: "2019-03-12T17:14:31.941"
> > > at com.test.DateTimeConverter.fromString(DateTimeConverter.java:32)
> > > at com.test.DateTimeConverter.fromString(DateTimeConverter.java:10)
> > > at org.apache.johnzon.mapper.internal.ConverterAdapter.to
> > > (ConverterAdapter.java:37)
> > > at org.apache.johnzon.mapper.internal.ConverterAdapter.to
> > > (ConverterAdapter.java:24)
> > > at
> > >
> > >
> >
> org.apache.johnzon.mapper.MappingParserImpl.toValue(MappingParserImpl.java:742)
> > > ... 141 more
> > > Caused by: java.text.ParseException: Unparseable date:
> > > "2019-03-12T17:14:31.941"
> > > at java.text.DateFormat.parse(DateFormat.java:366)
> > > at com.test.DateTimeConverter.fromString(DateTimeConverter.java:30)
> > > ... 145 more
> > >
> >
> >
> > --
> > Daniel "soro" Cunha
> > https://twitter.com/dvlc_
> >
>


-- 
Daniel "soro" Cunha
https://twitter.com/dvlc_

[GitHub] [tomee] ivanjunckes commented on issue #438: CXF 3.3.1 + MP Rest Client 1.2.1

[GitBox]

ivanjunckes commented on issue #438: CXF 3.3.1 + MP Rest Client 1.2.1
URL: https://github.com/apache/tomee/pull/438#issuecomment-472153254
 
 
   @j4fm hey James, is this ready to be merged? Or anything is still pending?


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


With regards,
Apache Git Services

RestClient issue - different apps with same client

[Ivan Junckes Filho]

Hey guys, there is a very annoying bug when using rest client from cxf
3.2.7 in TomEE M2. When 2 apps have the same client the server throws the
error below and shuts down?

Shouldn't we allow different apps in the same server use the same client?

*Error*
Caused by: org.apache.webbeans.exception.WebBeansException:
org.apache.webbeans.exception.DuplicateDefinitionException:
PassivationCapable bean id is not unique:
br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient
bean:SiemClassificacaoServiceClient, WebBeansType:THIRDPARTY,
Name:br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient,
API
Types:[br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient],
Qualifiers:[javax.enterprise.inject.Default,org.eclipse.microprofile.rest.client.inject.RestClient,javax.enterprise.inject.Any],
existing: SiemClassificacaoServiceClient, WebBeansType:THIRDPARTY,
Name:br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient,
API
Types:[br.com.gbrsistemas.crvirtual.classificacao.SiemClassificacaoServiceClient],
Qualifiers:[javax.enterprise.inject.Default,org.eclipse.microprofile.rest.client.inject.RestClient,javax.enterprise.inject.Any]
at
org.apache.webbeans.event.ObserverMethodImpl.notify(ObserverMethodImpl.java:371)
at
org.apache.webbeans.event.NotificationManager.invokeObserverMethod(NotificationManager.java:818)
at
org.apache.webbeans.event.NotificationManager.fireEvent(NotificationManager.java:714)
... 42 more

Re: Issue with @JohnzonConverter

[Ivan Junckes Filho]

I know Dani, that is the issue. But how to fix it? It should have been
converting to the right format.

On Tue, Mar 12, 2019 at 2:51 PM Daniel Cunha  wrote:

> Well.. seems that you convert expect that: dd/MM/ HH:mm:ss
> (
>
> https://github.com/ivanjunckes/convertertest/blob/master/src/main/java/com/test/DateTimeConverter.java#L17
> )
> But you are receiving that: 2019-03-12T17:14:31.941
>
> Em ter, 12 de mar de 2019 às 14:19, Ivan Junckes Filho <
> ivanjunc...@gmail.com> escreveu:
>
> > Hey guys, can anyone help me to understand why I am having issues with
> the
> > DateTimeConverter in my test?
> >
> > Here is a sample project with the issue.
> > https://github.com/ivanjunckes/convertertest
> >
> > If anyone could also help make this project work without the need for the
> > DateTimeConverter but actually using a configuration properties would be
> > great.
> >
> > *Error:*
> > Caused by: java.lang.IllegalArgumentException: java.text.ParseException:
> > Unparseable date: "2019-03-12T17:14:31.941"
> > at com.test.DateTimeConverter.fromString(DateTimeConverter.java:32)
> > at com.test.DateTimeConverter.fromString(DateTimeConverter.java:10)
> > at org.apache.johnzon.mapper.internal.ConverterAdapter.to
> > (ConverterAdapter.java:37)
> > at org.apache.johnzon.mapper.internal.ConverterAdapter.to
> > (ConverterAdapter.java:24)
> > at
> >
> >
> org.apache.johnzon.mapper.MappingParserImpl.toValue(MappingParserImpl.java:742)
> > ... 141 more
> > Caused by: java.text.ParseException: Unparseable date:
> > "2019-03-12T17:14:31.941"
> > at java.text.DateFormat.parse(DateFormat.java:366)
> > at com.test.DateTimeConverter.fromString(DateTimeConverter.java:30)
> > ... 145 more
> >
>
>
> --
> Daniel "soro" Cunha
> https://twitter.com/dvlc_
>

Re: Issue with @JohnzonConverter

[Daniel Cunha]

Well.. seems that you convert expect that: dd/MM/ HH:mm:ss
(
https://github.com/ivanjunckes/convertertest/blob/master/src/main/java/com/test/DateTimeConverter.java#L17
)
But you are receiving that: 2019-03-12T17:14:31.941

Em ter, 12 de mar de 2019 às 14:19, Ivan Junckes Filho <
ivanjunc...@gmail.com> escreveu:

> Hey guys, can anyone help me to understand why I am having issues with the
> DateTimeConverter in my test?
>
> Here is a sample project with the issue.
> https://github.com/ivanjunckes/convertertest
>
> If anyone could also help make this project work without the need for the
> DateTimeConverter but actually using a configuration properties would be
> great.
>
> *Error:*
> Caused by: java.lang.IllegalArgumentException: java.text.ParseException:
> Unparseable date: "2019-03-12T17:14:31.941"
> at com.test.DateTimeConverter.fromString(DateTimeConverter.java:32)
> at com.test.DateTimeConverter.fromString(DateTimeConverter.java:10)
> at org.apache.johnzon.mapper.internal.ConverterAdapter.to
> (ConverterAdapter.java:37)
> at org.apache.johnzon.mapper.internal.ConverterAdapter.to
> (ConverterAdapter.java:24)
> at
>
> org.apache.johnzon.mapper.MappingParserImpl.toValue(MappingParserImpl.java:742)
> ... 141 more
> Caused by: java.text.ParseException: Unparseable date:
> "2019-03-12T17:14:31.941"
> at java.text.DateFormat.parse(DateFormat.java:366)
> at com.test.DateTimeConverter.fromString(DateTimeConverter.java:30)
> ... 145 more
>


-- 
Daniel "soro" Cunha
https://twitter.com/dvlc_

Issue with @JohnzonConverter

[Ivan Junckes Filho]

Hey guys, can anyone help me to understand why I am having issues with the
DateTimeConverter in my test?

Here is a sample project with the issue.
https://github.com/ivanjunckes/convertertest

If anyone could also help make this project work without the need for the
DateTimeConverter but actually using a configuration properties would be
great.

*Error:*
Caused by: java.lang.IllegalArgumentException: java.text.ParseException:
Unparseable date: "2019-03-12T17:14:31.941"
at com.test.DateTimeConverter.fromString(DateTimeConverter.java:32)
at com.test.DateTimeConverter.fromString(DateTimeConverter.java:10)
at org.apache.johnzon.mapper.internal.ConverterAdapter.to
(ConverterAdapter.java:37)
at org.apache.johnzon.mapper.internal.ConverterAdapter.to
(ConverterAdapter.java:24)
at
org.apache.johnzon.mapper.MappingParserImpl.toValue(MappingParserImpl.java:742)
... 141 more
Caused by: java.text.ParseException: Unparseable date:
"2019-03-12T17:14:31.941"
at java.text.DateFormat.parse(DateFormat.java:366)
at com.test.DateTimeConverter.fromString(DateTimeConverter.java:30)
... 145 more

[GitHub] [tomee] otaviojava edited a comment on issue #422: Creates Unit test to CmpJpaConversion

[GitBox]

otaviojava edited a comment on issue #422: Creates Unit test to CmpJpaConversion
URL: https://github.com/apache/tomee/pull/422#issuecomment-472027903
 
 
   @dblevins done
   https://issues.apache.org/jira/browse/TOMEE-2492


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


With regards,
Apache Git Services

[GitHub] [tomee] otaviojava commented on issue #422: Creates Unit test to CmpJpaConversion

[GitBox]

otaviojava commented on issue #422: Creates Unit test to CmpJpaConversion
URL: https://github.com/apache/tomee/pull/422#issuecomment-472027903
 
 
   @dblevins done
   
https://issues.apache.org/jira/browse/TOMEE-2492?jql=text%20~%20%22CmpJpaConversion%22


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


With regards,
Apache Git Services

Re: [DISCUSS] version of our next javaee-api

[Mark Struberg]

 Oki great!
We should do a sigtest and if that is fine we can go on and release it as 
well.I've created a sigtest run script in various spec projects.Just look at 
json_1.1 for example.
LieGrue,strub


On Monday, 11 March 2019, 12:07:43 CET, Roberto Cortez 
 wrote:  
 
 Hum… JL mentioned that he merged 
https://github.com/apache/geronimo-specs/pull/14 


It seems to be there:
https://github.com/apache/geronimo-specs/tree/trunk/geronimo-security_1.0_spec 


Thanks!

Cheers,
Roberto

> On 9 Mar 2019, at 13:34, Mark Struberg  wrote:
> 
> I've only upgraded the various versions of our dependencies so far.I did not 
> add anything new. Not sure if we have that already.
> 
>    On Friday, 8 March 2019, 11:46:37 CET, Roberto Cortez 
> wrote:  
> 
> +1
> 
> Does this include the security api as well?
> 
>> On 8 Mar 2019, at 06:41, Mark Struberg  wrote:
>> 
>> David is right, next will be 8.0-1
>> 
>> 8.0 is for JavaEE8, -1 is the first update.
>> 
>> LieGrue,
>> Strub
>> 
>>> Am 07.03.2019 um 21:37 schrieb exabrial12 :
>>> 
>>> I'd stick to https://semver.org/ whenever possible :) This also helps Nexus
>>> and Maven make correct decisions.
>>> 
>>> 8.0.1 would be a bugfix, 8.1.0 would be new APIs are added but all old APIs
>>> remain, 9.0.0 would be old apis removed and possibly new stuff added.
>>> 
>>> 
>>> 
>>> 
>>> 
>>> --
>>> Sent from: http://tomee-openejb.979440.n4.nabble.com/TomEE-Dev-f982480.html