Hi Ivan, really nice, the content on README is much better now. Nice work man! Just pushing my +1 for your PR. :)
Em ter, 20 de nov de 2018 às 13:14, Ivan Junckes Filho < ivanjunc...@gmail.com> escreveu: > Hey guys, could you please review this? > > https://github.com/apache/tomee/pull/207 > > This is the project for counted. I will add more projects for other types > of metrics during the week. > > I would like to know if the README is good enough or if need more changes. > > Thanks. > > On Tue, Nov 20, 2018 at 11:30 AM Richard Monson-Haefel < > monsonhae...@gmail.com> wrote: > > > Hi David, > > > > This was really informative and useful. Thank you! > > > > Richard > > > > On Mon, Nov 19, 2018 at 6:43 PM David Blevins <david.blev...@gmail.com> > > wrote: > > > > > Agree with the README comments. Another great reference is the Simple > > > WebService example which has payloads. > > > > > > - > > > > > > https://github.com/apache/tomee/tree/master/examples/simple-webservice#inspecting-the-messages > > > > > > I think people new to MicroProfile metrics will want to see what the > > > metrics look like, so similarly showing payloads would be great. > > > > > > Often when I'm starting an bunch of examples, I try to plan a few in > > > advance so I can come up with a forward-thinking naming convention > should > > > be adjusted. There's an informal naming convention in there already > > which > > > is basically: > > > > > > - simple-<topic> // introduction > > > - <topic>-<subtopic1> // very specific lesson > > > - <topic>-<subtopic2> // very specific lesson > > > - <topic>-<subtopic3> // very specific lesson > > > > > > For example: > > > > > > - simple-webservice // introduction > > > - webservice-attachments > > > - webservice-handlerchain > > > - webservice-holder > > > > > > - cdi-basic // slightly bucks convention > > > - cdi-application-scope > > > - cdi-session-scope > > > - cdi-request-scope > > > - cdi-events > > > - cdi-interceptors > > > - cdi-decorators // this one doesn't actually exist!!! > > > contribution opportunity! :) > > > > > > My gut says "rest" is not the topic and shouldn't be first in the > > > directory name and probably doesn't need to be mentioned at all. The > > > second thought is there are very specific annotations and metric types, > > so > > > each of these could be an example. This might give you: > > > > > > - mp-metrics-basic || simple-mp-metrics // whichever convention > we > > > think is best > > > - mp-metrics-counted > > > - mp-metrics-guague > > > - mp-metrics-timed > > > > > > We've generally had a policy to attempt to keep examples as simple and > > > trivial as possible. For CDI we have one example for each scope > > > annotation. Doing the same with the metrics annotation would allow you > > to > > > focus on the nuances of that annotation. A few advantages would be: > > > > > > - search engine optimization. having the annotation in the page title > > > will put your content first when someone searches "mp metrics guague" > > > - people can send the links to each other with "this is the metric > type > > > you want for X task. Here's the simplest example. The other types are > > > ignorable for X" > > > - people don't have to digest all of MP Metrics at once. we typically > > > shoot for 5 minutes of "presentation time" per example. > > > > > > I vividly remember a presentation I did at JAX London where I talked > > about > > > OpenEJB for the first half, then just started showing examples for the > > > second half. Each five minutes. I showed what the room wanted to see. > > 30 > > > minutes go by and the session is technically over and it's lunch break, > > but > > > people aren't leaving. So I'm like, "do you want to see more?" and > > people > > > stay glued to their seat and keep asking for stuff. I showed another > > 20-30 > > > minutes of 5 minute examples before I basically told everyone to go eat > > as > > > the conference organizer who was standing in the back of the room the > > > entire time gave me the nod to force everyone get out of their seats. > It > > > was the biggest room and all the food and staff were just sitting > there. > > > > > > Anyway, that's the power of bite-sized examples. You can put them > > > together like legos and make any shape presentation you want. > > > > > > If you want to do a MicroProfile general introduction, you grab the > > > "simple foo" of each topic. If you want to do a dedicated MicroProfile > > > Metrics talk, you pick your favorite 5 or 8 mp-metrics-* examples. > > > > > > Doing it like that you can easily bang out a bunch of examples in a few > > > days or a week. Then have the ability to walk into almost any room and > > give > > > an ad-hoc MicroProfile presentation tailored to the people in front of > > you. > > > > > > > > > -- > > > David Blevins > > > http://twitter.com/dblevins > > > http://www.tomitribe.com > > > > > > > On Nov 19, 2018, at 12:24 PM, Jonathan Gallimore < > > > jonathan.gallim...@gmail.com> wrote: > > > > > > > > Great stuff! Do checkout these - they're a pretty good format to > > follow: > > > > > > > > > > https://github.com/apache/tomee/blob/master/examples/simple-stateless/README.md > > > > . > > > > > > > > I've merged your PR, and we can continue to iterate on it. Thanks for > > the > > > > contribution, I think its great to see some MicroProfile examples. I > > > found > > > > the examples really helpful when getting started with TomEE/OpenEJB > > > myself, > > > > and I'm sure folks will love seeing new examples for MicroProfile! > > > > > > > > Jon > > > > > > > > On Mon, Nov 19, 2018 at 8:12 PM Ivan Junckes Filho < > > > ivanjunc...@gmail.com> > > > > wrote: > > > > > > > >> Done Jon, I will keep improving it. > > > >> > > > >> On Mon, Nov 19, 2018 at 6:01 PM Jonathan Gallimore < > > > >> jonathan.gallim...@gmail.com> wrote: > > > >> > > > >>> Very cool. I'd suggest adding a README like the other examples > have, > > as > > > >>> that can be linked from the website, and they usually provide great > > > >>> documentation. > > > >>> > > > >>> Jon > > > >>> > > > >>> On Mon, 19 Nov 2018, 19:51 Ivan Junckes Filho < > ivanjunc...@gmail.com > > > >>> wrote: > > > >>> > > > >>>> Hey guys, just to let you know that I created a Microprofile > Metrics > > > >>>> Example under Tomee. > > > >>>> > > > >>>> I am still working on that and I will provide more test scenarios. > > > >>>> > > > >>>> Here is the PR if you want to review the work so far: > > > >>>> https://github.com/apache/tomee/pull/203 > > > >>>> > > > >>>> Feel free to merge if you think this can be done incrementally. > > > >>>> > > > >>> > > > >> > > > > > > > > > -- Daniel "soro" Cunha https://twitter.com/dvlc_
