You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tubemq.apache.org by GitBox <gi...@apache.org> on 2020/12/01 03:14:40 UTC

[GitHub] [incubator-tubemq] gosonzhang opened a new pull request #333: [TUBEMQ-436] Adjust Broker's HTTP API implementation

gosonzhang opened a new pull request #333:
URL: https://github.com/apache/incubator-tubemq/pull/333


   Adjust Broker's HTTP API implementation according to the new parameter check scheme:check the parameters through the field definition table of WEB API, unify the parameter field names and attribute values of each API interface, and facilitate interface expansion and maintenance


----------------------------------------------------------------
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.

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



[GitHub] [incubator-tubemq] gosonzhang merged pull request #333: [TUBEMQ-436] Adjust Broker's HTTP API implementation

Posted by GitBox <gi...@apache.org>.
gosonzhang merged pull request #333:
URL: https://github.com/apache/incubator-tubemq/pull/333


   


----------------------------------------------------------------
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.

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



[GitHub] [incubator-tubemq] codecov-io commented on pull request #333: [TUBEMQ-436] Adjust Broker's HTTP API implementation

Posted by GitBox <gi...@apache.org>.
codecov-io commented on pull request #333:
URL: https://github.com/apache/incubator-tubemq/pull/333#issuecomment-736190462


   # [Codecov](https://codecov.io/gh/apache/incubator-tubemq/pull/333?src=pr&el=h1) Report
   > Merging [#333](https://codecov.io/gh/apache/incubator-tubemq/pull/333?src=pr&el=desc) (b4d6433) into [TUBEMQ-430](https://codecov.io/gh/apache/incubator-tubemq/commit/c70b6eaba2beeaac8694b5f52ba5ffb3baf9e06f?el=desc) (c70b6ea) will **decrease** coverage by `0.05%`.
   > The diff coverage is `0.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/incubator-tubemq/pull/333/graphs/tree.svg?width=650&height=150&src=pr&token=1EUK92O9K2)](https://codecov.io/gh/apache/incubator-tubemq/pull/333?src=pr&el=tree)
   
   ```diff
   @@               Coverage Diff               @@
   ##             TUBEMQ-430    #333      +/-   ##
   ===============================================
   - Coverage          7.98%   7.92%   -0.06%     
   - Complexity          466     467       +1     
   ===============================================
     Files               254     254              
     Lines             27253   27372     +119     
     Branches           4446    4479      +33     
   ===============================================
   - Hits               2176    2170       -6     
   - Misses            24606   24731     +125     
     Partials            471     471              
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/incubator-tubemq/pull/333?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | [...e/tubemq/server/broker/web/BrokerAdminServlet.java](https://codecov.io/gh/apache/incubator-tubemq/pull/333/diff?src=pr&el=tree#diff-dHViZW1xLXNlcnZlci9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvdHViZW1xL3NlcnZlci9icm9rZXIvd2ViL0Jyb2tlckFkbWluU2VydmxldC5qYXZh) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | |
   | [.../tubemq/server/common/utils/WebParameterUtils.java](https://codecov.io/gh/apache/incubator-tubemq/pull/333/diff?src=pr&el=tree#diff-dHViZW1xLXNlcnZlci9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvdHViZW1xL3NlcnZlci9jb21tb24vdXRpbHMvV2ViUGFyYW1ldGVyVXRpbHMuamF2YQ==) | `2.47% <0.00%> (-0.79%)` | `8.00 <0.00> (ø)` | |
   | [.../java/org/apache/flume/sink/tubemq/TubemqSink.java](https://codecov.io/gh/apache/incubator-tubemq/pull/333/diff?src=pr&el=tree#diff-dHViZW1xLWNvbm5lY3RvcnMvdHViZW1xLWNvbm5lY3Rvci1mbHVtZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZmx1bWUvc2luay90dWJlbXEvVHViZW1xU2luay5qYXZh) | `51.42% <0.00%> (-4.00%)` | `14.00% <0.00%> (ø%)` | |
   | [.../producer/qltystats/DefaultBrokerRcvQltyStats.java](https://codecov.io/gh/apache/incubator-tubemq/pull/333/diff?src=pr&el=tree#diff-dHViZW1xLWNsaWVudC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvdHViZW1xL2NsaWVudC9wcm9kdWNlci9xbHR5c3RhdHMvRGVmYXVsdEJyb2tlclJjdlFsdHlTdGF0cy5qYXZh) | `46.09% <0.00%> (+0.39%)` | `22.00% <0.00%> (+1.00%)` | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-tubemq/pull/333?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/incubator-tubemq/pull/333?src=pr&el=footer). Last update [c70b6ea...b4d6433](https://codecov.io/gh/apache/incubator-tubemq/pull/333?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


----------------------------------------------------------------
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.

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