Skip to content

#3729 Inbox pattern add support for json#3956

Merged
iancooper merged 10 commits into
BrighterCommand:masterfrom
ArmanPopat:AddJsonSupportForInboxMessage
Jan 28, 2026
Merged

#3729 Inbox pattern add support for json#3956
iancooper merged 10 commits into
BrighterCommand:masterfrom
ArmanPopat:AddJsonSupportForInboxMessage

Conversation

@ArmanPopat

@ArmanPopat ArmanPopat commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

Added support for persisting message payloads using the database’s native JSON column types where available for the Inbox pattern. This includes JSON/JSONB support for PostgreSQL and JSON support for MySQL.

A new flag is added to the RelationalDatabaseConfiguration to allow for this.

@CLAassistant

CLAassistant commented Dec 29, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

codescene-delta-analysis[bot]

This comment was marked as outdated.

codescene-delta-analysis[bot]

This comment was marked as outdated.

Comment thread tests/Paramore.Brighter.PostgresSQL.Tests/Inbox/PostgresJsonbInboxAsyncTest.cs Outdated
Comment thread tests/Paramore.Brighter.PostgresSQL.Tests/Inbox/PostgresJsonbInboxTest.cs Outdated
Comment thread tests/Paramore.Brighter.PostgresSQL.Tests/Inbox/PostgresJsonInboxAsyncTest.cs Outdated
Comment thread tests/Paramore.Brighter.PostgresSQL.Tests/Inbox/PostgresJsonInboxTest.cs Outdated
@iancooper iancooper added 2 - In Progress feature request .NET Pull requests that update .net code labels Dec 31, 2025
Co-authored-by: Rafael Lillo <7280959+lillo42@users.noreply.github.com>
codescene-delta-analysis[bot]

This comment was marked as outdated.

codescene-delta-analysis[bot]

This comment was marked as outdated.

codescene-delta-analysis[bot]

This comment was marked as outdated.

@iancooper

Copy link
Copy Markdown
Member

It looks good @ArmanPopat. I just want to check on the build, etc. before we sign this one off

codescene-delta-analysis[bot]

This comment was marked as outdated.

@codescene-delta-analysis codescene-delta-analysis Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gates Passed
4 Quality Gates Passed

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

@iancooper

Copy link
Copy Markdown
Member

@ArmanPopat Seeing if I can make the build more reliable. We suffer from timing issues if we get co-located with noisy neighbors. But I will watch and merge

@iancooper iancooper merged commit 9266ef4 into BrighterCommand:master Jan 28, 2026
43 of 49 checks passed
@iancooper

Copy link
Copy Markdown
Member

Merged @ArmanPopat We will get a build out soon

DevJonny pushed a commit to DevJonny/Brighter that referenced this pull request Feb 28, 2026
…and#3956)

* BrighterCommand#3729  Inbox pattern add support for json

* BrighterCommand#3729  Inbox pattern add support for json

* Remove commented code

* Apply suggestions from code review - whitespace changes

Co-authored-by: Rafael Lillo <7280959+lillo42@users.noreply.github.com>

* Adding ADR

---------

Co-authored-by: Rafael Lillo <7280959+lillo42@users.noreply.github.com>
Co-authored-by: Ian Cooper <ian_hammond_cooper@yahoo.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Done feature request .NET Pull requests that update .net code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants