Skip to content

Conversation

pniedzielski
Copy link
Collaborator

@pniedzielski pniedzielski commented Jul 28, 2025

There are some legacy types in mqbconfm that don't directly relate to domain configuration, such as Request and Response types.

The Request type can be removed with no issue.

The Response and DomainResolver types are used internally in mqba::ConfigProvider and mqba::DomainResolver, and can be replaced with simple bsl::strings.

TODO

  • One last pass on ConfigProvider
  • Do same refactor to DomainResolver
  • Remove DomainDefinition type.
  • Regenerate code from XSD.

@pniedzielski pniedzielski added the A-Broker Area: C++ Broker label Jul 28, 2025
@pniedzielski pniedzielski force-pushed the cleanup/mqbconf branch 2 times, most recently from b00124a to d291228 Compare July 30, 2025 19:29
Signed-off-by: Patrick M. Niedzielski <[email protected]>
Signed-off-by: Patrick M. Niedzielski <[email protected]>
…cted

This shows that the log was incorrect.

Signed-off-by: Patrick M. Niedzielski <[email protected]>
Signed-off-by: Patrick M. Niedzielski <[email protected]>
Signed-off-by: Patrick M. Niedzielski <[email protected]>
Signed-off-by: Patrick M. Niedzielski <[email protected]>
Signed-off-by: Patrick M. Niedzielski <[email protected]>
Signed-off-by: Patrick M. Niedzielski <[email protected]>
Copy link

@bmq-oss-ci bmq-oss-ci bot left a comment

Choose a reason for hiding this comment

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

Build 2954 of commit 9fb4307 has completed with FAILURE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Broker Area: C++ Broker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant