You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by GitBox <gi...@apache.org> on 2022/10/21 12:38:37 UTC

[GitHub] [skywalking-java] wu-sheng commented on pull request #362: Polish up rabbitmq-5.x plugin to fix missing broker tag on consumer side

wu-sheng commented on PR #362:
URL: https://github.com/apache/skywalking-java/pull/362#issuecomment-1286907121

   > We could provide NotBlankAssertor in test tool project , or set the value to keyValueBuilder even if it's null
   
   Usually, we don't suggest post tag with empty value, so, we only have `not null` for now. If you feel necessary, `not blank` should be fine, but only for string field or tag.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org