Andreas Braun

Results 235 comments of Andreas Braun

Note: I'll have to revisit the failing Windows Build tasks for PHP 7.4 and 8.0 separately - deferring the merge until those have been taken care of.

@dependabot ignore this dependency. I've created GODRIVER-3556 to manually update the dependency once we require Go 1.23.

The database name that is part of the connection string is the default auth database, which can also be given in the `authSource` URI option. It is not a "default...

@johnnyshields the Ruby team generally doesn't merge up their branches, but cherry-picks to older branches. Can you elaborate on what the reason is for merging 9.0 into master?

Hi, there are two separate answers here: one for root documents, one for embedded documents. For embedded documents, you can disable this behaviour through the `rename_embedded_id_field` connection option as shown...

@SMFloris unfortunately, the name `id` is hardcoded as an identifier in various places, despite model classes having a dedicated field to indicate the primary key name. We have created pull...

@hans-thomas in general, the changes look good. I am a little bit worried about the BC impact of this change, as any fields that previously used an array cast will...

@GromNaN I believe the request is to allow configuring a cache pool directly, e.g. for the metadata cache. From what I can see, we have not yet ported this change...

I can take a look at the clang-format issue. It looks like there is a memory leak in some tests: ``` [2025/07/28 09:44:20.753] FAILURE: (FAILED) [2025/07/28 09:44:20.753] -- [2025/07/28 09:44:20.753]...

@danog the windows failure is unrelated, and the first impression from Evergreen is that the memory leak is no more. As for the remaining `zend_hash_destroy` usage, I have no objection...