Re: Docker images

2019-06-06 Thread Otávio Gonçalves de Santana
t;> Yes - thanks for the reminder! >>> >>> On Thu, Jun 6, 2019 at 12:15 PM j4fm wrote: >>> >>>> Would somebody be able to take a look at updating Docker hub with M3? >>>> >>>> >>>> >>>> -- >>>> Sent from: >>>> http://tomee-openejb.979440.n4.nabble.com/TomEE-Dev-f982480.html >>>> >>> -- Otávio Gonçalves de Santana twitter: http://twitter.com/otaviojava site: *http://about.me/otaviojava <http://about.me/otaviojava>*

Re: TOMEE-2492 - Creates Unit test to CmpJpaConversion

2019-04-15 Thread Otávio Gonçalves de Santana
The goal of this issue is to create a unit test at CmpJpaConversion and also based on these test make sure that any change does not change the desired behavior from the class. On Fri, Apr 12, 2019 at 1:05 PM César Hernández Mendoza < cesargu...@gmail.com> wrote: > Hi Otavio, > > I assigned to you

Re: TOMEE-2492 - Creates Unit test to CmpJpaConversion

2019-04-15 Thread Otávio Gonçalves de Santana
I also fired a ticket: https://issues.apache.org/jira/browse/TOMEE-2492 On Mon, Apr 15, 2019 at 8:42 AM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > The goal of this issue is to create a unit test at CmpJpaConversion and > also based on these test make sure that any

Re: Idea of an M3

2019-04-09 Thread Otávio Gonçalves de Santana
+1 On Tue, Apr 9, 2019 at 9:33 AM Thomas Andraschko < andraschko.tho...@gmail.com> wrote: > We should at least wait for the next BVal and OpenJPA version, they are > currently in the release vote :) > > Am Di., 9. Apr. 2019 um 13:39 Uhr schrieb j4fm >: > > > Hey, sorry I've been out for 4 weeks

Re: @OpenAPIDefinition not working

2019-04-01 Thread Otávio Gonçalves de Santana
Hey, I could fix and I created a PR: https://github.com/apache/tomee/pull/454 Could someone help me on how to create a test on this? Thank you. On Mon, Apr 1, 2019 at 1:01 PM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > > I tried a workaround to ignore at CdiScan

Re: @OpenAPIDefinition not working

2019-04-01 Thread Otávio Gonçalves de Santana
s. My next step is to try at the *OpenEJBLifecycle*. On Fri, Mar 29, 2019 at 5:45 PM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > > > I created the TomEEOpenAPIExtension that wrapper the > GeronimoOpenAPIExtension. > > My question is, how can I shut

Re: @OpenAPIDefinition not working

2019-03-29 Thread Otávio Gonçalves de Santana
not work. I need to ignore the GeronimoOpenAPIExtension, to avoid the duplicated creation of Bean at CDI. On Fri, Mar 29, 2019 at 9:52 AM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > Unfortunately, this approach does not work. > > > ProxyFactory proxyFactory

Re: @OpenAPIDefinition not working

2019-03-29 Thread Otávio Gonçalves de Santana
cation foo = (Application)aClass.newInstance(); ((ProxyObject)foo).setHandler(mi); Class aClass1 = foo.getClass(); I'll try another strategy. On Thu, Mar 28, 2019 at 2:39 PM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > One possible solution is the Javassist &

Re: @OpenAPIDefinition not working

2019-03-28 Thread Otávio Gonçalves de Santana
cation) factory.create(new Class[0], new Object[0], handler); That is possible, but it does not look good. I'll try another option. Also, we need to include this library on the Server. Thoughts? On Thu, Mar 28, 2019 at 10:45 AM Otávio Gonçalves de Santana < osant...@tomitribe.com&

Re: @OpenAPIDefinition not working

2019-03-28 Thread Otávio Gonçalves de Santana
The InternalApplication is required, because, at RESTService it needs to add classes as a wrapper. My first thought is to create a proxy, however, Application is a class and not an interface. On Thu, Mar 28, 2019 at 9:43 AM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote:

Re: @OpenAPIDefinition not working

2019-03-28 Thread Otávio Gonçalves de Santana
mar de 2019 às 18:24, Otávio Gonçalves de Santana < > osant...@tomitribe.com> escreveu: > > > My maven configuration: > > > > http://maven.apache.org/POM/4.0.0"; > > xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; > &

Re: MicroProfile Integration in Plus and Plume

2019-03-28 Thread Otávio Gonçalves de Santana
at org.apache.cxf.jaxrs.JAXRSServerFactoryBean.create(JAXRSServerFactoryBean.java:157) ... 75 more On Thu, Mar 28, 2019 at 7:53 AM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > I tried this one, but I had some regressions issues. > I'm still working on one way to fix this. > Qu

Re: MicroProfile Integration in Plus and Plume

2019-03-28 Thread Otávio Gonçalves de Santana
from Apache? On Wed, Mar 27, 2019 at 3:07 PM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > > Hey. > I've tested the OpeAPI. > That still an error when we put description at an Application. > > > @ApplicationPath("api")@OpenAPIDefi

Re: @OpenAPIDefinition not working

2019-03-27 Thread Otávio Gonçalves de Santana
PIResponse(responseCode = "200", description = "Successful, returning the value") }) @Produces(MediaType.TEXT_PLAIN) public Response getConfigValue(@PathParam("key") String key) { return Response.ok(key).build(); } } On Wed, Mar 27, 2019 at 5:11

Re: @OpenAPIDefinition not working

2019-03-27 Thread Otávio Gonçalves de Santana
Hey Ivan. I tried something, and I saw the same issue. It seems this by-pass does not work. On Mon, Feb 11, 2019 at 11:18 AM Ivan Junckes Filho wrote: > It didn't seem to work for me adding the property to system.properties. Any > ideas what I am doing wrong? > > On Sat, Feb 9, 2019 at 10:21 AM

Re: MicroProfile Integration in Plus and Plume

2019-03-27 Thread Otávio Gonçalves de Santana
Hey. I've tested the OpeAPI. That still an error when we put description at an Application. @ApplicationPath("api")@OpenAPIDefinition(info = @Info( title = "Example application", version = "1.0.0", contact = @Contact( name = "Otavio", email

Re: How can I help

2019-03-18 Thread Otávio Gonçalves de Santana
Welcome Alex, On Mon, Mar 18, 2019 at 5:44 PM Alex Rabelo Ferreira < alexrabeloferreira@gmail.com> wrote: > Thanks for the explanation Daniel. > > So, I will code every sunday for the community. First, I will translate the > examples of security and after, I will work on a project to GOJAVA.

Creates Unit test to CmpJpaConversion

2019-03-13 Thread Otávio Gonçalves de Santana
The goal of this issue is to create a unit test at CmpJpaConversion and also based on these test make sure that any change does not change the desired behavior from the class. Ref: https://github.com/apache/tomee/pull/422

Re: Building with Java 11

2019-03-04 Thread Otávio Gonçalves de Santana
de mar de 2019 às 11:54, Otávio Gonçalves de Santana < > osant...@tomitribe.com> escreveu: > > > Now, I got a new error: > > > > Illegal reflective access by > org.apache.openejb.loader.BasicURLClassPath$4 > > > > > (file:/tmp/temp14662460932513

Re: Building with Java 11

2019-03-04 Thread Otávio Gonçalves de Santana
That looks illegal access is forbidden at Java 11. I'm working on it. On Mon, Mar 4, 2019 at 11:54 AM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > > Now, I got a new error: > > Illegal reflective access by org.apache.openejb.loader.BasicURLC

Re: Building with Java 11

2019-03-04 Thread Otávio Gonçalves de Santana
Now, I got a new error: Illegal reflective access by org.apache.openejb.loader.BasicURLClassPath$4 (file:/tmp/temp14662460932513193953dir/root/lib/openejb-loader-8.0.0-SNAPSHOT.jar) to field java.net.URLClassLoader.ucp On Mon, Mar 4, 2019 at 10:49 AM Otávio Gonçalves de Santana < os

Re: Building with Java 11

2019-03-04 Thread Otávio Gonçalves de Santana
hem as part of TomEE > itself. Its desirable to run the itests on both Java 8 and 11 to see if > there are any issues. > > Jon > > On Mon, Mar 4, 2019 at 1:27 PM Otávio Gonçalves de Santana < > osant...@tomitribe.com> wrote: > > > Hey JonG. > > Could you expla

Re: Building with Java 11

2019-03-04 Thread Otávio Gonçalves de Santana
On Fri, Mar 1, 2019 at 5:19 PM Jonathan Gallimore < jonathan.gallim...@gmail.com> wrote: > If that's just for itests, I'd be inclined to try creating a profile to add > that where the Java version is >8. > > What do you think? > > Jon > > On Fri,

Re: Apache TomEE channel in the-asf.slack.com

2019-03-01 Thread Otávio Gonçalves de Santana
Done On Fri, Mar 1, 2019 at 5:54 PM j4fm wrote: > Me too please > > > > -- > Sent from: > http://tomee-openejb.979440.n4.nabble.com/TomEE-Dev-f982480.html >

Re: Apache TomEE channel in the-asf.slack.com

2019-03-01 Thread Otávio Gonçalves de Santana
Hey Ivan, done On Fri, Mar 1, 2019 at 4:52 PM Ivan Junckes Filho wrote: > It seems it forces you to have a apache email or to be invited. I can't > join, can anyone invite me? > > On Fri, Mar 1, 2019 at 4:51 PM Daniel Cunha wrote: > > > Hi Gabriel, > > > > Yes! > > > > Em sex, 1 de mar de 2019

Re: Building with Java 11

2019-03-01 Thread Otávio Gonçalves de Santana
Hey guys I'm working to make the tests compatible with both Java 8 and Java 11 version. Right now, I'm at the failover project that has this issue: java.lang.IllegalAccessException: class org.apache.openejb.loader.BasicURLClassPath c

Re: Building with Java 11

2019-03-01 Thread Otávio Gonçalves de Santana
Hey JonG, What do you think if you move this branch to the TomEE project? On Wed, Jan 9, 2019 at 10:40 AM Jean-Louis Monteiro < jlmonte...@tomitribe.com> wrote: > Great job Jon. > I did some tests so, so we should be able to get something green hopefully > soon > -- > Jean-Louis Monteiro > http:/

Re: TOMEE-2477 - Upgrade .gitignore to ignore files generated by test.

2019-02-27 Thread Otávio Gonçalves de Santana
+1 Nice Daniel. On Wed, Feb 27, 2019 at 1:15 PM Daniel Cunha wrote: > Hi folks, > > Ticket: https://issues.apache.org/jira/browse/TOMEE-2477 > > PR: https://github.com/apache/tomee/pull/425 > > Please review it and merge. > > Thank you. > > -- >

Mockito or any Mock framework at TomEE

2019-02-27 Thread Otávio Gonçalves de Santana
Hello everyone. I have a question. Do we have any Mock framework such as JMockito at TomEE? I would like to create a test at openejb-core, but I did not find any dependency on it.

Re: PRs

2019-02-26 Thread Otávio Gonçalves de Santana
Thank you JonG. On Tue, Feb 26, 2019 at 8:49 AM Jean-Louis Monteiro < jlmonte...@tomitribe.com> wrote: > Good job. Thanks Jon > -- > Jean-Louis Monteiro > http://twitter.com/jlouismonteiro > http://www.tomitribe.com > > > On Tue, Feb 26, 2019 at 12:38 PM Jonathan Gallimore < > jonathan.gallim...@

Re: [ANNOUNCE] Apache TomEE 8.0.0-M2 Released

2019-01-31 Thread Otávio Gonçalves de Santana
That is also available at docker hub: https://hub.docker.com/_/tomee On Thu, Jan 31, 2019 at 9:12 AM Roberto Cortez wrote: > Hi Frankie, > > I’ve sent the email to both users and dev. > > Cheers, > Roberto > > > On 31 Jan 2019, at 06:16, Frankie wrote: > > > > Just found it: TomEE Users mailing

Re: [VOTE] Release Apache TomEE 8.0.0 MILESTONE 2

2019-01-26 Thread Otávio Gonçalves de Santana
+1 nice Roberto On Sat, Jan 26, 2019, 05:10 Jean-Louis Monteiro Thank you Roberto. > I'll review over the weekend or Monday morning first time. > -- > Jean-Louis Monteiro > http://twitter.com/jlouismonteiro > http://www.tomitribe.com > > > On Sat, Jan 26, 2019 at 3:26 AM César Hernández Mendoza <

Re: CmpJpaConversion does not work in an Ear package [WIP]

2019-01-24 Thread Otávio Gonçalves de Santana
Thank you for the Help, Jonathan. I've finished this PR, thank you On Mon, Jan 21, 2019 at 12:15 PM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > Here what I’ve tried: > >- Use the @EBJ injection at the ActorBean >- Use the @Inject at the ActorBean

Re: PR review and merge

2019-01-24 Thread Otávio Gonçalves de Santana
if any of these PR’s would break it, so better to be safe :) > > > On 24 Jan 2019, at 12:05, Otávio Gonçalves de Santana < > osant...@tomitribe.com> wrote: > > > > Hello everyone, please don't forget these PRs > > > > On Wed, Jan 23, 2019 at 8:01 AM

Re: PR review and merge

2019-01-24 Thread Otávio Gonçalves de Santana
Hello everyone, please don't forget these PRs On Wed, Jan 23, 2019 at 8:01 AM Bruno Baptista wrote: > Hi, > > Can one of the committers please take a look at these PRs? > > https://github.com/apache/tomee/pull/377 > https://github.com/apache/tomee/pull/376 > https://github.com/apache/tomee/pull/

Re: Another 8.0.0 milestone release

2019-01-23 Thread Otávio Gonçalves de Santana
+1 On Wed, Jan 23, 2019 at 8:48 AM Roberto Cortez wrote: > There are a few MP issues that we need to sort out regarding plume and > plus (if we want to push MP into plus and plume). I’ll explain those in the > other thread. We need to fix them or drop plus and plume support for this > release. >

Re: CmpJpaConversion does not work in an Ear package [WIP]

2019-01-21 Thread Otávio Gonçalves de Santana
NRE, DATA_TYPE: CHARACTER VARYING, CHARACTER_MAXIMUM_LENGTH: 255")); Assert.assertTrue(output.contains("TABLE_NAME: MOVIE, COLUMN_NAME: MOVIE_NAME, DATA_TYPE: CHARACTER VARYING, CHARACTER_MAXIMUM_LENGTH: 250")); final String[] split = output.split("\r?\n"); Asse

Re: CmpJpaConversion does not work in an Ear package [WIP]

2019-01-18 Thread Otávio Gonçalves de Santana
)); System.out.println(war.toString(true)); System.out.println(archive.toString(true)); return archive; } @Test public void checkCmpJpaEntityORMMappings() throws Exception { Assert.assertNotNull(actorBean); } } On Thu, Jan 17, 2019 at 5:02 PM Otávio Gonçalves de Santana &

Re: CmpJpaConversion does not work in an Ear package [WIP]

2019-01-17 Thread Otávio Gonçalves de Santana
I created a solution, could you review it, please? I'm working on a test to it. On Thu, Jan 17, 2019 at 10:09 AM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > > > I put these codes at CmpJpaConversion: > > > URL resource = CmpJpaConversion.class.

Re: CmpJpaConversion does not work in an Ear package [WIP]

2019-01-17 Thread Otávio Gonçalves de Santana
e classloader, so what's on the classpath when > we try and read this file? > > Jon > > On Thu, Jan 17, 2019 at 11:44 AM Otávio Gonçalves de Santana < > osant...@tomitribe.com> wrote: > > > Given an EAR package that has an EJB jar with a persistenc

CmpJpaConversion does not work in an Ear package [WIP]

2019-01-17 Thread Otávio Gonçalves de Santana
Given an EAR package that has an EJB jar with a persistence.xml within a mapping-file element. The CmpJpaConversion does not find this reference because when i

Re: Optmizes try catch #323

2019-01-09 Thread Otávio Gonçalves de Santana
>> Hi Otávio, > >> > >> I will take a look. > >> > >> Cheers > >> > >> Bruno Baptista > >> https://twitter.com/brunobat_ > >> > >> > >> On 09/01/19 10:16, Otávio Gonçalves de Santana wrote: &g

Re: Optmizes try catch #323

2019-01-09 Thread Otávio Gonçalves de Santana
Thank you for the review guys. Could someone move it forward it, please? On Fri, Dec 28, 2018 at 3:47 PM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > >- > >Optimizes the try using the try-with-resources that allows us to >declare resources to b

Optmizes try catch #323

2018-12-28 Thread Otávio Gonçalves de Santana
- Optimizes the try using the try-with-resources that allows us to declare resources to be used in a try block with the assurance that the resources will be closed when after execution of that block. An ARM automatic resource management by the JVM. - The use of multi-catch th

Re: TomEE logo poll

2018-12-28 Thread Otávio Gonçalves de Santana
+1 On Fri, Dec 28, 2018 at 3:38 PM Khalid HORRI wrote: > +1 > > On Fri, Dec 28, 2018 at 6:29 PM Daniel Dias Dos Santos < > daniel.dias.analist...@gmail.com> wrote: > > > +1 > > -- > > > > *Daniel Dias dos Santos* > > Java Developer > > SouJava & JCP Member > > GitHub: https://github.com/Daniel-D

Re: Ejemplos en Español

2018-12-28 Thread Otávio Gonçalves de Santana
+1 On Wed, Dec 26, 2018 at 9:11 AM Daniel Cunha wrote: > +1 > > Em qua, 26 de dez de 2018 7:34 AM, Bruno Baptista escreveu: > > > +1 > > > > Bruno Baptista > > https://twitter.com/brunobat_ > > > > > > On 24/12/18 22:35, David Blevins wrote: > > > I was talking with Hillmer on twitter about pot

Re: [GitHub] tomee pull request #283: Tomee 2330 backport 7 0

2018-12-19 Thread Otávio Gonçalves de Santana
Nice, please don't forget the backport to version 7.1.x https://github.com/apache/tomee/pull/285 On Wed, Dec 19, 2018 at 7:53 PM asfgit wrote: > Github user asfgit closed the pull request at: > > https://github.com/apache/tomee/pull/283 > > > --- >

Re: CMP/JPA mapping not accounting for in persistence.xml

2018-12-17 Thread Otávio Gonçalves de Santana
The second PR: https://github.com/apache/tomee/pull/285 On Mon, Dec 17, 2018 at 10:21 AM Bruno Baptista wrote: > Great! Thanks! > > Bruno Baptista > https://twitter.com/brunobat_ > > > On 17/12/18 12:19, Otávio Gonçalves de Santana wrote: > > Thank you Bruno, I

Re: CMP/JPA mapping not accounting for in persistence.xml

2018-12-17 Thread Otávio Gonçalves de Santana
Thank you Bruno, I created a PR to fix missing header On Mon, Dec 17, 2018 at 9:46 AM Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > Hello, I created the backport to version 7.0.x: > https://github.com/apache/tomee/pull/283 > > On Wed, Dec 5, 2018 at 4:21 PM Gur

Re: CMP/JPA mapping not accounting for in persistence.xml

2018-12-17 Thread Otávio Gonçalves de Santana
Hello, I created the backport to version 7.0.x: https://github.com/apache/tomee/pull/283 On Wed, Dec 5, 2018 at 4:21 PM Gurkan Erdogdu wrote: > For the ORB case, there are places where ORB class is imported > (openejb-core and openejb-client). For java 11, this will probably not > compile and ne

Re: Is TomEE ready to ship for Santa? - Conducting pre-xmas releases: 7.0.6, 7.1.1, 8.0.0-M2

2018-12-13 Thread Otávio Gonçalves de Santana
+1 On Thu, Dec 13, 2018, 10:37 Roberto Cortez Hi Martin, > > Yes, my idea was to push for another Milestone release once we stabilise > the build. > > Cheers, > Roberto > > > On 13 Dec 2018, at 12:16, Wiesner, Martin < > martin.wies...@hs-heilbronn.de> wrote: > > > > Hey guys, > > > > Richard (Z,

Re: Adds getModuleId method as a fallback when getJarLocation is null at

2018-12-13 Thread Otávio Gonçalves de Santana
AM Jonathan Gallimore < > > jonathan.gallim...@gmail.com> wrote: > > > > > Awesome, thank you! Merged. > > > > > > Jon > > > > > > On Thu, Dec 13, 2018 at 10:56 AM Otávio Gonçalves de Santana < > > > osant...@tomitribe.com> wrote:

Adds getModuleId method as a fallback when getJarLocation is null at

2018-12-13 Thread Otávio Gonçalves de Santana
It PR adds the getModuleId method as a fallback when getJarLocation is null at CmpJpaConversion, this way it keeps the desired behavior and fixes the when the getJarLocation is null such as the CheckMissingClassTest. https://github.com/apache/tomee/pull/274

Removes unnecessary boxing/unboxing process

2018-12-11 Thread Otávio Gonçalves de Santana
This PR removes unnecessary boxing and unboxing at the code base within Tomee container. This PR has the goal to both remove code line and allow a slight performance improvement. Also, provides to the compiler do it when it is indispensable. Ref: https://github.com/apache/tomee/pull/271

Re: Move to Gitbox?

2018-12-11 Thread Otávio Gonçalves de Santana
+1 On Tue, Dec 11, 2018 at 1:47 PM Matthew Broadhead wrote: > +1 > good idea > > On 11/12/2018 12:44, Jonathan Gallimore wrote: > > Hi Folks > > > > It appears that the git-wip service is going to be retired, and the > > repositories are going to be migrated over to Gitbox - > > https://gitbox.a

Optimize string operations

2018-12-07 Thread Otávio Gonçalves de Santana
This PR has the goal to optimize String operations in the module container. - The first operation is instead of using the method isEmpty instead of use string.equals("") - The second operation changes a String with one characater to a char type. @Warmup(iterations = 5, time = 1)@Measu

Re: MicroProfile 2.0

2018-12-07 Thread Otávio Gonçalves de Santana
JL, I like baby steps. Maybe, keep MP 1.3, release a TomEE 8 RC-2, then upgrade the version to MP 2.0. On Fri, Dec 7, 2018 at 1:40 PM Jean-Louis Monteiro wrote: > Let's revive this discussion ... > > We are working on build stability (see other thread). > I was digging into an issue where there

Re: Approving Pull Requests

2018-12-07 Thread Otávio Gonçalves de Santana
That is a nice point. I usually do this, I just keep one PR active and just open a second one when the first is merged. I think that became easier to review and merge process. On Thu, Dec 6, 2018 at 10:24 AM Richard Monson-Haefel < monsonhae...@gmail.com> wrote: > I went through and gave my appr

Re: Improve performance using Collections in TomEE container

2018-12-05 Thread Otávio Gonçalves de Santana
Thank you Bruno, let's wait to a commiter move it forward :) On Wed, Dec 5, 2018 at 2:36 PM Bruno Baptista wrote: > Hi All, > > I reviewed Otávio's PR and it looks ok to merge. > > Cheers > > Bruno Baptista > https://twitter.com/brunobat_ > > > On

Update the API code

2018-12-05 Thread Otávio Gonçalves de Santana
Hello everyone, I'm studying the code, mostly the container module. It seems that in the master we support Java 8. My question: Does make sense to create PRs to update some APIs to a modern way? I mean, e.g.: for (int i = 0; i < rules.length; i++) { rules[i].validate(appModule); } To for (Va

Re: JMS Documentation PR

2018-12-04 Thread Otávio Gonçalves de Santana
Ok, I'll do. On Tue, Dec 4, 2018 at 2:25 PM David Blevins wrote: > > On Dec 4, 2018, at 4:19 AM, Otávio Gonçalves de Santana < > osant...@tomitribe.com> wrote: > > > > Hey David, I did more modifications and improving it folling the > > recomendations

Improve performance using Collections in TomEE container

2018-12-04 Thread Otávio Gonçalves de Santana
This PR has the Goal to improve performance and save memory on the collections code. To get this goal, we went to different improvements: 1) Use the Constructor in the collection instead of add Set set = new HashSet<>(); set.addAll(Arrays.asList("alpha", "beta", "gamma")); Set set = new HashSet<

Re: JMS Documentation PR

2018-12-04 Thread Otávio Gonçalves de Santana
Hey David, I did more modifications and improving it folling the recomendations from JonG. IMHO: That is enough to the first step documentaiton. Thank you for remember this PR :), I forget completelly. On Mon, Dec 3, 2018 at 5:00 PM David Blevins wrote: > Hey Otavio, > > I see a PR from you that

Re: Example for concurrency utilities

2018-12-03 Thread Otávio Gonçalves de Santana
+1 On Thu, Nov 29, 2018 at 8:27 PM Richard Monson-Haefel < monsonhae...@gmail.com> wrote: > +1 > > On Thu, Nov 29, 2018 at 1:22 PM César Hernández Mendoza < > cesargu...@gmail.com> wrote: > > > +1 > > > > El jue., 29 nov. 2018 a las 11:55, Bruno Baptista () > > escribió: > > > > > Hi folks, > > >

Avoid String in a loop

2018-11-26 Thread Otávio Gonçalves de Santana
The reason to prefer StringBuilder is that both + and concat create a new object every time you call them (provided the right-hand side argument is not empty). This can quickly add up to a lot of objects, almost all of which are completely unnecessary. public class Main{ public static void mai

Re: Creates an unmarshal that does not filter to JavaEE namespace

2018-11-21 Thread Otávio Gonçalves de Santana
html > > I don't know if creating an example wouldn't be simpler than creating a > test inside the container. > > Bruno Baptista > https://twitter.com/brunobat_ > > > On 21/11/18 15:43, Otávio Gonçalves de Santana wrote: > > Thank you, let me put more cont

Re: Creates an unmarshal that does not filter to JavaEE namespace

2018-11-21 Thread Otávio Gonçalves de Santana
Baptista wrote: > I'll take a look at the PR too. > > Bruno Baptista > https://twitter.com/brunobat_ > > > On 21/11/18 13:40, Otávio Gonçalves de Santana wrote: > > Ok thank you Jean-Louis and Jonathan Gaillimore. > > > > On Wed, Nov 21, 2018 at 11:25 AM Jean-L

Re: Creates an unmarshal that does not filter to JavaEE namespace

2018-11-21 Thread Otávio Gonçalves de Santana
.com/jlouismonteiro > http://www.tomitribe.com > > > On Wed, Nov 21, 2018 at 2:22 PM Jonathan Gallimore < > jonathan.gallim...@gmail.com> wrote: > > > Sure. I'll take a look. > > > > On Wed, Nov 21, 2018 at 12:58 PM Otávio Gonçalves de Santana < >

Re: Creates an unmarshal that does not filter to JavaEE namespace

2018-11-21 Thread Otávio Gonçalves de Santana
d-superclass>,<{ > http://java.sun.com/xml/ns/persistence/orm}package> > > My guess is its been broken a while, and is broken in master, 7.1.x, 7.0.x, > 1.7.x and maybe even earlier than that. > > I'll merge this PR in and we'll see what the build looks like on the

Is there someone looking for Microprofile Reactive at TomEE?

2018-11-20 Thread Otávio Gonçalves de Santana
Hello everyone, at the Eclipse Microprofile world there are a lot of implementations that TomEE covers, but, I did not find the implementation to microprofile-reactive-streams . Does someone know if this Job has started at the TomEE side? I

Uses Boolean.valueof instaed of new instance

2018-11-20 Thread Otávio Gonçalves de Santana
While I was studying the code, I found at SingletonEjbObjectHandler.java that they are creating a new Boolean instance instead of using the Boolean.valueOf. method. The boolean wrapper has a singleton instance to both true and false. However, it will use when the code uses the Boolean.valueOf metho

Re: Creates an unmarshal that does not filter to JavaEE namespace

2018-11-19 Thread Otávio Gonçalves de Santana
un into it before. I'll pick up your PR > and > > > review. > > > > > > Cheers! > > > > > > Jon > > > > > > On Mon, Nov 19, 2018 at 11:33 AM Otávio Gonçalves de Santana < > > > osant...@tomitribe.com> wrote: > >

Re: Creates an unmarshal that does not filter to JavaEE namespace

2018-11-19 Thread Otávio Gonçalves de Santana
com> wrote: > Hey Otavio > > Thanks for posting. At what point do you run into this exception (your > stacktrace shows the unmarshal, but not where its called from). If you > have a PR, shoot a link here and I'll be happy to take a look. > > Jon > > On Mon, No

Re: Creates an unmarshal that does not filter to JavaEE namespace

2018-11-19 Thread Otávio Gonçalves de Santana
led from). If you > have a PR, shoot a link here and I'll be happy to take a look. > > Jon > > On Mon, Nov 19, 2018 at 10:31 AM Otávio Gonçalves de Santana < > osant...@tomitribe.com> wrote: > > > Hi all. > > > > We found an error when using

Creates an unmarshal that does not filter to JavaEE namespace

2018-11-19 Thread Otávio Gonçalves de Santana
Hi all. We found an error when using CMP; when the container starts, it returns an unmarshal error: at com.sun.xml.bind.v2.runtime.unmarshaller.UnmarshallingContext.handleEvent(UnmarshallingContext.java:662) at com.sun.xml.bind.v2.runtime.unmarshaller.Loader.reportError(Loader.java:258

Re: [VOTE] Release Apache TomEE 7.0.5 (round 2)

2018-07-19 Thread Otávio Gonçalves de Santana
+1 On Thu, Jul 19, 2018 at 8:24 AM, Jonathan Gallimore < jonathan.gallim...@gmail.com> wrote: > Here's my +1. > > Jon > > On Tue, Jul 10, 2018 at 7:26 PM, Jonathan Gallimore < > jonathan.gallim...@gmail.com> wrote: > > > Hi Everyone, > > > > Here is the second roll of TomEE 7.0.5. Please can you

Re: tomee git commit: replacing 7.0.5 by 7.0.6-SNAPSHOT

2018-07-18 Thread Otávio Gonçalves de Santana
Maybe there is a miss understanding, on my side, but the idea would not be after the 7.0.5 the project move this branch to master: https://github.com/apache/tomee/tree/fb_tomee8 then start to work in Java EE8, isn't? On Tue, Jul 17, 2018 at 12:20 PM, Thiago Veronezi wrote: > https://issues.apach

Re: [VOTE] Merge Pull Request 123 - MicroProfile JWT support

2018-03-29 Thread Otávio Gonçalves de Santana
+ 1, Let's merge and move it forward. On Thu, Mar 29, 2018 at 3:12 PM, Bruno Baptista wrote: > + 1, Let's merge. > > If later someone wants to move it elsewhere in the future, that's fine. > > While people figure out how to do that, at least we will have something > working for the users. > > Ch

Re: Implementing Microprofile JWT

2018-02-02 Thread Otávio Gonçalves de Santana
Guys, I have a question: Why not a project to each implementation? This way I can use just a specific if I want also. On Fri, Feb 2, 2018 at 7:44 AM, Romain Manni-Bucau wrote: > Hi JL, > > Microprofile apache effort is hosted in geronimo and John already spoke > about it I think. Would probably

Improves MDB performance with instance pool

2017-12-27 Thread Otávio Gonçalves de Santana
Ivan Junckes and I have been working to improve performance with MDB pools. This goal of this proposal is to improve performance in the message-driven bean creation using a pool of 10 objects (default value). The strategy is to keep these objects live so that they can be reused instead of every t

Re: Improving the JMS documentation at the website

2017-07-14 Thread Otávio Gonçalves de Santana
PR created: https://github.com/apache/tomee-site-generator/pull/4 On Thu, Jul 13, 2017 at 2:25 PM, Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > > The resource for using JMS setting from both file resource and annotation. > > Eg: > > > >

Re: Improving the JMS documentation at the website

2017-07-13 Thread Otávio Gonçalves de Santana
. > > Jon > > On Thu, Jul 13, 2017 at 4:48 PM, Otávio Gonçalves de Santana < > osant...@tomitribe.com> wrote: > > > Hey folks > > > > I'm writing documentation to Improve the JMS at the website. > > This enhancement has the following topics

Improving the JMS documentation at the website

2017-07-13 Thread Otávio Gonçalves de Santana
Hey folks I'm writing documentation to Improve the JMS at the website. This enhancement has the following topics: - Understanding the on TomEE JMS - JMS - Message - Message Types - JMS Advanced - JMS Advanced - Activation Configuration - Configurations

Re: Adds an option to ignore mdb.activation fields

2017-07-12 Thread Otávio Gonçalves de Santana
t; | > LinkedIn <https://www.linkedin.com/in/rmannibucau> | JavaEE Factory > <https://javaeefactory-rmannibucau.rhcloud.com> > > 2017-07-12 15:04 GMT+02:00 Otávio Gonçalves de Santana < > osant...@tomitribe.com>: > > > String containerId = ejbDeployment.getCo

Re: Adds an option to ignore mdb.activation fields

2017-07-12 Thread Otávio Gonçalves de Santana
es. Tests around bean > properties overriding container properties would be good too, as would > testing both bean and container properties being overridden by system > properties. > > Cheers > Jon > > On 2017-07-12 12:42 (+0100), Otávio Gonçalves de Santana < > osant...@tomit

Re: Adds an option to ignore mdb.activation fields

2017-07-12 Thread Otávio Gonçalves de Santana
There is other problem: ejbDeployment.getContainerId(); //it returns null On Wed, Jul 12, 2017 at 10:04 AM, Otávio Gonçalves de Santana < osant...@tomitribe.com> wrote: > String containerId = ejbDeployment.getContainerId(); > > It returns a container id, > How can I return

Re: Adds an option to ignore mdb.activation fields

2017-07-12 Thread Otávio Gonçalves de Santana
in/rmannibucau> | JavaEE Factory > <https://javaeefactory-rmannibucau.rhcloud.com> > > 2017-07-12 14:29 GMT+02:00 Otávio Gonçalves de Santana < > osant...@tomitribe.com>: > > > 1) Getter removed, thanks > > 2) I took a look at that. As far as I can see, we don

Re: Adds an option to ignore mdb.activation fields

2017-07-12 Thread Otávio Gonçalves de Santana
running the tests. If it looks ok, I'll merge it, while Otavio works on > > the > > > container-based config in another patch. Please shout if you have any > > > objections. > > > > > > Otavio, let us know if you need any help or guidance on the cont

Re: Adds an option to ignore mdb.activation fields

2017-07-12 Thread Otávio Gonçalves de Santana
sed > settings! > > Jon > > On Thu, Jul 6, 2017 at 9:44 PM, Otávio Gonçalves de Santana < > osant...@tomitribe.com> wrote: > > > The problem > > > > The configuration for MDB activation properties allow any key/value to be > > specified. At present on t

Re: Adds an option to ignore mdb.activation fields

2017-07-07 Thread Otávio Gonçalves de Santana
ntainer/ > > > > openejb-core/src/main/java/org/apache/openejb/config/ > > > > ActivationConfigPropertyOverride.java#L94 > > > > using .container.? > > > > > > > > +1 if ~so (just wanted to avoid a misunderstanding and get a > completel

Adds an option to ignore mdb.activation fields

2017-07-06 Thread Otávio Gonçalves de Santana
The problem The configuration for MDB activation properties allow any key/value to be specified. At present on the 1.7.x branch, the server will fail to deploy the application if the activation property is not present on the activation spec class. This becomes painful in a scenario where more tha