Posted by jonloh on September 17, 2012 at 3:59am
7 followers
| Project: | Mongodb |
| Version: | 7.x-1.0-rc2 |
| Component: | Cache |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Issue Summary
Deprecated function: MongoBinData::__construct(): The default value for type will change to 0 in the future. Please pass in '2' explicitly. in DrupalMongoDBCache->set() (line 131 of ......../sites/all/modules/mongodb/mongodb_cache/mongodb_cache.inc).
I believe this doesnt seem to be much of an issue, but it does affect the performance a little as it keeps on flooding the watchdog with every request.
Comments
#1
Note: this warning is new in version 1.2.11 of the extension.
Going further, I wonder whether there would not be more to gain from alway using a
MongoBinData::BYTE_ARRAYinstead of routing each string data throughdrupal_validate_utf8()as the check currently does.This probably needs performance testing, but that would be a followup issue.
#2
The last submitted patch, 0001-Issue-1786330-MongoBinData-__construct-needed-an-exp.patch, failed testing.
#3
+1 flooding the watchdog on each cache operation is bad.
#4
Applied patch from #1 and it seems to be not flooding the logs anymore. Should consider this to be comitted :)
Thanks fgm!
#5
#1: 0001-Issue-1786330-MongoBinData-__construct-needed-an-exp.patch queued for re-testing.
#6
The last submitted patch, 0001-Issue-1786330-MongoBinData-__construct-needed-an-exp.patch, failed testing.