http://ayende.comAyende @ RahienCopyright (C) Ayende Rahien 2004 - 2021 (c) 202660thinking out loud.. if I'm looking at queries for my app, I want to know that it runs the first time properly, and uses cached query for subsequent calls... Not regarding your UI, I'd envision it in a tree view or some way of drilling down. Would it be possible to have a timeline view of the executed queries and a relational view (same queries together, possibly different params)? http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment8http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment8Tue, 06 Jan 2009 16:23:28 GMTTuna, Yes, but you fork it, if you are the only one that needs what you implement. Otherwise, you first make sure that the feature you implement is in sync with where the piece of software is going overall. http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment7http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment7Tue, 06 Jan 2009 09:03:43 GMTKrzysztof, isn't it the way oss work in general? If you need something, you implement something. http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment6http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment6Tue, 06 Jan 2009 09:01:19 GMTAyende, It's not what I feared it would be. You're right, these changes benefit anyone using NHibernate, or at least thehy don't hurt anybody. What I feared, was that you would start adding changes that would support NHProf, at the expense of making NHibernate less robust, harder to use, add no value for anyone else but NHProf, or hurting it overall in some other way. http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment5http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment5Tue, 06 Jan 2009 08:57:31 GMTVery nice feature :) I know this is shrink wrapped software, but how about drawing a feature freeze for this iteration and getting R1 out the door? Then, give us an R2 in 2-4 weeks with more features :) By the way, I LOVE the fact that you can modify NHibernate to make a feature possible. I see this as an advantage of Open Source software. I realise there's a potential for abuse there too, especially if the modifier benefits financially from the changes. I also think your NHProf site is looking very concise, and cool. Have you considered linking to the site on each blog post, and also stating that the tool is available for download? (I realised this already, but I want to see a link regardless). http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment4http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment4Tue, 06 Jan 2009 08:23:43 GMTKrzysztof, Take a look at the changes, then tell me what you think. [nhibernate.svn.sourceforge.net/.../nhibernate](http://nhibernate.svn.sourceforge.net/viewvc/nhibernate?view=rev&revision=3976) [nhibernate.svn.sourceforge.net/.../nhibernate](http://nhibernate.svn.sourceforge.net/viewvc/nhibernate?view=rev&revision=3979) They also benefit who ever is using NH without the profiler. http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment3http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment3Tue, 06 Jan 2009 08:02:34 GMT I admire your work overall, and with Castle/NHibernate in particular. However I'm not sure I like the idea of changing NHibernate solely to make a 3rd party tool happy. http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment2http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment2Tue, 06 Jan 2009 06:44:55 GMTvery glad to see this feature as well as the last two that were introduced ;) i'd leave the tests in these posts... it doesn't hurt anyone and it's interesting to at least some people http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment1http://ayende.com/3797/nh-prof-new-feature-the-query-cache#comment1Tue, 06 Jan 2009 06:29:35 GMT