Skip to content

Fix NullReferenceException when connected to RavenDB cloud#5112

Merged
mauroservienti merged 3 commits into
masterfrom
ravendb-dirty-memory-check-NRE
Sep 10, 2025
Merged

Fix NullReferenceException when connected to RavenDB cloud#5112
mauroservienti merged 3 commits into
masterfrom
ravendb-dirty-memory-check-NRE

Conversation

@ramonsmits

Copy link
Copy Markdown
Member

Fix NullReferenceException when connected to RavenDB cloud which does not return memory information.

@mauroservienti mauroservienti left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than, maybe, more readable, is is null any better?

Comment thread src/ServiceControl.Audit.Persistence.RavenDB/MemoryInformationRetriever.cs Outdated
…Retriever.cs

Co-authored-by: Mauro Servienti <mauro.servienti@gmail.com>
@ramonsmits

Copy link
Copy Markdown
Member Author

Other than, maybe, more readable, is is null any better?

It is, null == c habbit still haunts me after 30+ years ;-)

Comment thread src/ServiceControl.Persistence.RavenDB/MemoryInformationRetriever.cs Outdated
@mauroservienti
mauroservienti merged commit fa7ed5a into master Sep 10, 2025
32 checks passed
@mauroservienti
mauroservienti deleted the ravendb-dirty-memory-check-NRE branch September 10, 2025 10:22
@bartekwasielak bartekwasielak added this to the 6.8.0 milestone Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants