Skip to content

Comments

Add AMQPLib plugin (RabbitMQ)#34

Merged
kezhenxu94 merged 4 commits intoapache:masterfrom
tom-pytel:master
Mar 4, 2021
Merged

Add AMQPLib plugin (RabbitMQ)#34
kezhenxu94 merged 4 commits intoapache:masterfrom
tom-pytel:master

Conversation

@tom-pytel
Copy link
Contributor

I will see about doing the test tomorrow if I am feeling masochistic enough...

@tom-pytel tom-pytel force-pushed the master branch 3 times, most recently from 79b0903 to 804b083 Compare March 2, 2021 12:51
@kezhenxu94 kezhenxu94 added this to the 0.2.0 milestone Mar 2, 2021
@tom-pytel tom-pytel force-pushed the master branch 5 times, most recently from 3a5d54c to e16767f Compare March 2, 2021 13:44
@tom-pytel
Copy link
Contributor Author

tom-pytel commented Mar 2, 2021

Ok, I got this mostly done except that some process does not exit (loops trying to exit), can you have a look?

EDIT: Am waiting on this to move on to the Entry / Exit span overhaul.

@tom-pytel
Copy link
Contributor Author

I have some important internal fixes waiting on this getting merged, how about I remove the failing test and send it up on a separate branch so that this can get merged without the test and I can push the fixes and the RabbitMQ test can be handled later?

@kezhenxu94
Copy link
Member

I have some important internal fixes waiting on this getting merged, how about I remove the failing test and send it up on a separate branch so that this can get merged without the test and I can push the fixes and the RabbitMQ test can be handled later?

Good for me. Go ahead.

@kezhenxu94 kezhenxu94 merged commit 25944ed into apache:master Mar 4, 2021
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.

2 participants