Skip to content

[#1846] Add monitoring (events) to template rendering...#792

Open
PerfectCarl wants to merge 18 commits intoplayframework:1.3.xfrom
PerfectCarl:play13_monitoring
Open

[#1846] Add monitoring (events) to template rendering...#792
PerfectCarl wants to merge 18 commits intoplayframework:1.3.xfrom
PerfectCarl:play13_monitoring

Conversation

@PerfectCarl
Copy link
Copy Markdown
Contributor

#779... JPQL queries and JPA operations.

See this ticket

Basically, it will help profiling modules to display database and rendering time

Like this

@PerfectCarl
Copy link
Copy Markdown
Contributor Author

Fixed merge. With documentation (events.textile)

@cloudbees-pull-request-builder
Copy link
Copy Markdown

play-1-3-x-pull-requests #258 FAILURE
Looks like there's a problem with this pull request

@PerfectCarl
Copy link
Copy Markdown
Contributor Author

any news on this ?

@pepite
Copy link
Copy Markdown
Contributor

pepite commented Oct 27, 2014

Quite frankly, this should be done within a module and not within the core. I think you should be able to do that with the current API and by extending the JPA module.

@pepite pepite closed this Oct 27, 2014
@PerfectCarl
Copy link
Copy Markdown
Contributor Author

@pepite Well, as a matter of fact, using the existing JPA module didn't yield interesting results and the code was starting to be a fragile mess.
Leveraging that existing framework feature seemed more elegant.

Trust me, I really wished to do that using the module feature. Maybe there is a way that I didn't consider. Have you any pointer?

By the way, asking to document the patch and not accepting it is rude. Next time could you make your mind beforehand?

@xael-fry xael-fry reopened this May 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants