- ๐ฌ๐ทGreece n.antonopoulou
The latest patch was applied successfully and the issue was resolved to my project.
- ๐ฌ๐ทGreece christosgeorgiadis
Updated the cherry-picked patch to rc15
- @prem-suthar opened merge request.
- ๐ฎ๐ณIndia prem suthar Ahemdabad- Gujrat , Jodhpur - Rajsthan
prem suthar โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia chandansha
Please Review the code and let me know if i missed anything.
Thanks!! - @chandansha opened merge request.
- First commit to issue fork.
- First commit to issue fork.
- First commit to issue fork.
- First commit to issue fork.
- ๐ฎ๐นItaly mondrake ๐ฎ๐น
@daffie well, that's why I suggest using events, that gives us the possibility to extend/override how they are processed. At least conceptually, the MongoDB driver could implement a subscriber/listener for the
ExecuteMethodEnsuringSchemaEvent
, make it so that it processes the event before core'sSchemaRequestSubscriber
does, do its alternative processing of the schema request, and stop propagation of the event so that core no longer processes it afterwards. Of course, needs to be proven in practice. - ๐ณ๐ฑNetherlands daffie
@mondrake: It looks like a beautiful solution. My problem is that the solution will not work with MongoDB. The solution relies on that the database will throw an exception when the table does not exists. However that is not what MongoDB does. MongoDB just creates a table when one does not exists on an insert. A table in MongoDB is called a collection. It will be a table with no validation added to make sure that it only allows data in the right form being added. See: https://www.mongodb.com/docs/manual/reference/method/db.collection.inser....
The reason I started to work on this issue was to add a hook to allow the database driver to change the table schema. The database driver for MongoDB stores timestamps as MongoDB\BSON\UTCDateTime instead of an integer value as is done by the by core supported databases.
If we could make it to work for MongoDB that would be great. I will need some time to think how I can make it to work with MongoDB. If you have some ideas or suggestions, then please speak up.
- ๐ฎ๐นItaly mondrake ๐ฎ๐น
Filed ๐ Explore injecting the event dispatcher in the database service Active as a possible follow up.
- ๐ฎ๐นItaly mondrake ๐ฎ๐น
Now
Flood\DatabaseBackend
implemenrs #166, and if this is OK we need to convert the rest where possible (i.e. calls to dynamic queries extendingQuery
).Pausing now to let reviewing happen.
- ๐ฎ๐นItaly mondrake ๐ฎ๐น
- ๐จ๐ญSwitzerland znerol
I'm not too happy keeping an unbound amount of state inside the page cache middleware. The original approach in #3029373: PageCache::getCacheId() sometimes uses different cache IDs during get and set, causing unnecessary cache misses for some requests โ would be compatible with n main requests (look at the patches before comment
#13
.