Skip to content

Changed log message ID type from String to Integer #5163

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Changed log message ID type from String to Integer #5163

wants to merge 1 commit into from

Conversation

corentinaltepe
Copy link

I noticed an error in the log message documentation with mongo v4.4.
I was receiving log message with an "id" being an Integer, and not a String. Example:

{
  ...
  "id": 123456,
  ...
}

So I offer to update documentation.

Porting to other branches?

I have not tested any other version of mongo, nor compared with the doc for other branches. I would expect this change should be ported to other branches but I couldn't evaluate which ones.

Related work where the inconsistency was noticed: Mongo2Go/Mongo2Go#114

@andf-mongodb
Copy link
Contributor

Thank you, you are correct. I've merged this in to primary v4.4 branch via #5336

@corentinaltepe corentinaltepe deleted the fix/type/4.4 branch May 12, 2021 06:33
mongo-cr-bot pushed a commit that referenced this pull request Nov 2, 2023
* DOCSP-33304 Clarify Mirrored Reads

* *

* punctuation
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 23, 2025
…le behavior (#5163)

* doc tweak

* MONGOID-5195 MONGOID-5237 reword compare_time_by_ms description

* MONGOID-5151 MONGOID-5233 broken_alias_handling

* MONGOID-5183 MONGOID-5236 broken_and

* MONGOID-5186 MONGOID-5229 broken_scoping

* MONGOID-4755 MONGOID-5239 broken_aggregables

* MONGOID-5126 MONGOID-5232 broken_triple_equals

* more triple equals tests

* MONGOID-5206 MONGOID-5235 broken_updates

* MONGOID-5162 MONGOID-5234 object_id_as_json_oid

* change to legacy_triple_equals

* order options alphabetically

Co-authored-by: Oleg Pudeyev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants