DDC-3761: Entity Cache Key save bug

[doctrinebot]

Jira issue originally created by user Mark.Ragazzo:

When using memcached for caching query or entities, queries will be performed each time, since EntityCacheKey use space in its key. I can submit PR if needed. Let me know if it is needed to be fixed in other places, however i looked through the code and have not found any other places

[doctrinebot]

Comment created by Mark.Ragazzo:

any news on this one? Should i submit PR or is there any internals issues?

[doctrinebot]

Comment created by @FabioBatSilva:

That seems to be the issue,
Please fell free to send a PR

[doctrinebot]

Comment created by Mark.Ragazzo:

fixed it with simple replacing spaces to dots, in this PR #1423. We already use this in production with Second Level Cache, however we replaced whole key with md5() for simplicity. Also it may not be question for you but why there is no checks for key length for cache? Since default policy for SCL is to rely on namespaces when generate keys it can easily go out of 250 bytes for memcached, since it invoke a lot of prefixing and so on.

[doctrinebot]

Comment created by Mark.Ragazzo:

http://www.doctrine-project.org/jira/browse/[DDC-3771](http://www.doctrine-project.org/jira/browse/DDC-3771)

[lcobucci]

Closing as invalid since it should be solved on the memcache(d) provider in doctrine/cache.