This repository has been archived on 2023-08-20. You can view files and clone it, but cannot push or open issues or pull requests.

Fabien Potencier a00c1f5927 merged branch Wotre/profiler_mongodb (PR #1784) ... Commits ------- 41b7a19 Updated the tests so that tests will be marked as skipped when there is no MongoDB server present! 233c7db Updated the code to follow the symfony coding standards 7b24de5 Updated the code to follow the symfony coding standard using stof his remarks fbcbdde - Fixed a small bug - Updated some phpdoc 00fdfec Added a MongoDbProfilerStorage engine Discussion ---------- [2.1] [HttpKernel] MongoDb storage for Profiler As a documentbased database like MongoDB is [supposedly fantastic in logging](http://blog.mongodb.org/post/172254834/mongodb-is-fantastic-for-logging) I implemented a storage engine for the profiler that should enable us to use this database as storage for this. Activate it using this way: framework: profiler: dsn: mongodb://user:pass@location/database/collection --------------------------------------------------------------------------- by stof at 2011/07/24 11:23:06 -0700 btw, the MongoDB session storage has already be rejected from the core so this should probably be moved to the DoctrineMongoDBBundle (even if it uses only the PHP extension and not Doctrine). @fabpot thoughts about this ? --------------------------------------------------------------------------- by Wotre at 2011/07/24 11:52:56 -0700 Just my personal opinion, if it is prefered that way I will move this into the DoctrineMongoDBBundle. While it is reasonable to bundle all Mongo related things together, I do believe that in the case of logging we want to avoid as many depencies as possible. Some exceptions can occur pretty early inside the framework, and it would be a shame if those aren't logged because this layer is written on top of doctrine. I'm not exactly familliar enough with the symfony internals as I only started using it a few days ago, but I can imagine that this can make a difference with some exceptions. --------------------------------------------------------------------------- by stof at 2011/07/24 11:59:10 -0700 I don't ask you to use Doctrine in this code. It is fine to use the extension directly if it is enough. Btw, the profiler is *not* used early. :) --------------------------------------------------------------------------- by Wotre at 2011/07/26 10:45:05 -0700 So... Any final remark whether this should be moved to [the DoctrineMongoDBBundle](https://github.com/symfony/DoctrineMongoDBBundle) or not? If it has to be moved, any comment on where in that bundle this should be put? Also, if it has to be moved, how can we arrange the configuration using DI? Currently I've put a line in the FrameworkExtension file to use this engine for everything with a $dsn starting with mongodb; I imagine this kind of ugly depency can't really exist between the FrameworkBundle and another one. Although it seems completely illogical to me, I will move it, but I do need some directions on how to elegantly do this... --------------------------------------------------------------------------- by stof at 2011/07/26 11:03:04 -0700 @fabpot what do you think ? --------------------------------------------------------------------------- by stof at 2011/09/04 01:28:48 -0700 @fabpot what do you think about the place where this should be done ?		2011-09-06 07:53:28 +02:00
..
FileProfilerStorageTest.php	[HttpKernel] Update tests for FileProfilerStorage	2011-08-29 15:35:28 +02:00
MongoDbProfilerStorageTest.php	Updated the tests so that tests will be marked as skipped when there is no MongoDB server present!	2011-09-04 16:56:09 +02:00
ProfilerTest.php	refactored Profiler class	2011-05-30 22:25:25 +02:00
SqliteProfilerStorageTest.php	[Tests] Fixed typos	2011-06-12 20:52:53 +00:00