Skip to content

feat(amqp10): implement queue declaration - #237

Open
sharathb5 wants to merge 6 commits into
feature-amqp10from
190-provider-interface-implement-declarequeue
Open

feat(amqp10): implement queue declaration#237
sharathb5 wants to merge 6 commits into
feature-amqp10from
190-provider-interface-implement-declarequeue

Conversation

@sharathb5

@sharathb5 sharathb5 commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Add AMQP 1.0 queue declaration with compatible queue options and coverage.

@sharathb5
feat(amqp10): implement queue declaration
b84e22d 
Signed-off-by: Sharath Bhattiprolu <sharathjb23@gmail.com>
@sharathb5 sharathb5 linked an issue Oct 8, 2026 that may be closed by this pull request
Provider Interface: implement declareQueue #190
Open
@sharathb5
sharathb5 changed the base branch from main to feature-amqp10 October 8, 2026 17:14
@sharathb5
fix(amqp10): address lint errors
c815079 
Signed-off-by: Sharath Bhattiprolu <sharathjb23@gmail.com>
@sharathb5
sharathb5 marked this pull request as ready for review October 8, 2026 17:33
rsperl
rsperl reviewed Oct 8, 2026
View reviewed changes
Comment thread internal/provider/connectors/rabbitmq/amqp10/amqp10_provider.go Outdated
rsperl
rsperl reviewed Oct 8, 2026
View reviewed changes
Comment thread internal/provider/connectors/rabbitmq/amqp10/amqp10_provider.go
rsperl
rsperl reviewed Oct 8, 2026
View reviewed changes
Comment thread internal/provider/connectors/rabbitmq/amqp10/amqp10_provider.go Outdated
@sharathb5
fix(amqp10): address PR comments
e5f19c1 
Signed-off-by: Sharath Bhattiprolu <sharathjb23@gmail.com>
rsperl
rsperl reviewed Oct 8, 2026
View reviewed changes
Comment thread tests/integration/amqp10_provider_connect_test.go Outdated
rsperl
rsperl reviewed Oct 8, 2026
View reviewed changes
Comment thread tests/integration/amqp10_provider_connect_test.go Outdated
@sharathb5
fix(amqp): address PR comments
9b0e4a1 
Signed-off-by: Sharath Bhattiprolu <sharathjb23@gmail.com>
rsperl
rsperl reviewed Oct 12, 2026
View reviewed changes
Comment thread internal/provider/connectors/amqp/mgmt_rest.go Outdated
rsperl
rsperl reviewed Oct 12, 2026
View reviewed changes
Comment thread internal/provider/connectors/amqp/mgmt_rest.go
rsperl
rsperl reviewed Oct 12, 2026
View reviewed changes
Comment thread internal/provider/connectors/amqp/mgmt_rest.go
rsperl
rsperl reviewed Oct 12, 2026
View reviewed changes
Comment thread internal/provider/connectors/rabbitmq/amqp10/amqp10_provider.go Outdated
@sharathb5
fix(amqp10): address queue declaration review
7d59ae6 
Signed-off-by: Sharath Bhattiprolu <sharathjb23@gmail.com>
rsperl
rsperl reviewed Oct 13, 2026
View reviewed changes
Comment thread internal/provider/connectors/rabbitmq/amqp10/amqp10_provider.go
@sharathb5
test(amqp10): isolate broker connection integration test
71428e9 
Signed-off-by: Sharath Bhattiprolu <sharathjb23@gmail.com>
rsperl
rsperl reviewed Oct 13, 2026
View reviewed changes
Comment thread internal/provider/connectors/rabbitmq/amqp10/amqp10_provider.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rsperl rsperl rsperl approved these changes
@dlawregiets dlawregiets Awaiting requested review from dlawregiets dlawregiets is a code owner
@bithckr bithckr Awaiting requested review from bithckr bithckr is a code owner
@dhunt3800 dhunt3800 Awaiting requested review from dhunt3800 dhunt3800 is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Provider Interface: implement declareQueue

2 participants

@sharathb5 @rsperl