You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by ted-ross <gi...@git.apache.org> on 2018/08/10 14:39:39 UTC

[GitHub] qpid-dispatch pull request #358: DISPATCH-930 - Initial incomplete implement...

GitHub user ted-ross opened a pull request:

    https://github.com/apache/qpid-dispatch/pull/358

    DISPATCH-930 - Initial incomplete implementation for Edge-Router:

      - Added configuration elements for edge router mode and edge-uplink connection role
      - Added iterator changes needed for interpreting edge summary addresses and uplinks
      - Added test configuration for an edge with two interiors
      - Added new system tests for edge router functionality

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ted-ross/qpid-dispatch DISPATCH-930-6

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/qpid-dispatch/pull/358.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #358
    
----
commit 5ea16062fdce74bb92bab38351fa00dc38b477c7
Author: Ted Ross <tr...@...>
Date:   2018-08-10T14:34:37Z

    DISPATCH-930 - Initial incomplete implementation for Edge-Router:
      - Added configuration elements for edge router mode and edge-uplink connection role
      - Added iterator changes needed for interpreting edge summary addresses and uplinks
      - Added test configuration for an edge with two interiors
      - Added new system tests for edge router functionality

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[GitHub] qpid-dispatch issue #358: DISPATCH-930 - Initial incomplete implementation f...

Posted by codecov-io <gi...@git.apache.org>.
Github user codecov-io commented on the issue:

    https://github.com/apache/qpid-dispatch/pull/358
  
    # [Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/358?src=pr&el=h1) Report
    > Merging [#358](https://codecov.io/gh/apache/qpid-dispatch/pull/358?src=pr&el=desc) into [master](https://codecov.io/gh/apache/qpid-dispatch/commit/6576b3e9d0113cd691fcd092a4e36076579498d4?src=pr&el=desc) will **decrease** coverage by `0.14%`.
    > The diff coverage is `75.78%`.
    
    [![Impacted file tree graph](https://codecov.io/gh/apache/qpid-dispatch/pull/358/graphs/tree.svg?width=650&height=150&src=pr&token=rk2Cgd27pP)](https://codecov.io/gh/apache/qpid-dispatch/pull/358?src=pr&el=tree)
    
    ```diff
    @@            Coverage Diff             @@
    ##           master     #358      +/-   ##
    ==========================================
    - Coverage    84.5%   84.35%   -0.15%     
    ==========================================
      Files          69       69              
      Lines       15729    15842     +113     
    ==========================================
    + Hits        13292    13364      +72     
    - Misses       2437     2478      +41
    ```
    
    
    | [Impacted Files](https://codecov.io/gh/apache/qpid-dispatch/pull/358?src=pr&el=tree) | Coverage Δ | |
    |---|---|---|
    | [src/router\_core/agent\_connection.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2Nvbm5lY3Rpb24uYw==) | `74.34% <ø> (ø)` | :arrow_up: |
    | [src/router\_core/route\_control.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlX2NvbnRyb2wuYw==) | `95.83% <ø> (ø)` | :arrow_up: |
    | [src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=) | `92.15% <0%> (-7.12%)` | :arrow_down: |
    | [src/python\_embedded.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-c3JjL3B5dGhvbl9lbWJlZGRlZC5j) | `77.59% <100%> (ø)` | :arrow_up: |
    | [tests/parse\_test.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-dGVzdHMvcGFyc2VfdGVzdC5j) | `68.06% <100%> (ø)` | :arrow_up: |
    | [tests/field\_test.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-dGVzdHMvZmllbGRfdGVzdC5j) | `84.75% <100%> (+0.66%)` | :arrow_up: |
    | [src/router\_node.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9ub2RlLmM=) | `93.73% <100%> (+0.21%)` | :arrow_up: |
    | [src/router\_core/route\_tables.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlX3RhYmxlcy5j) | `76.31% <100%> (+0.12%)` | :arrow_up: |
    | [src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=) | `93.06% <57.14%> (-2.36%)` | :arrow_down: |
    | [src/iterator.c](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree#diff-c3JjL2l0ZXJhdG9yLmM=) | `87.71% <97.95%> (+1.1%)` | :arrow_up: |
    | ... and [4 more](https://codecov.io/gh/apache/qpid-dispatch/pull/358/diff?src=pr&el=tree-more) | |
    
    ------
    
    [Continue to review full report at Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/358?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/qpid-dispatch/pull/358?src=pr&el=footer). Last update [6576b3e...6a30871](https://codecov.io/gh/apache/qpid-dispatch/pull/358?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).



---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[GitHub] qpid-dispatch issue #358: DISPATCH-930 - Initial incomplete implementation f...

Posted by ted-ross <gi...@git.apache.org>.
Github user ted-ross commented on the issue:

    https://github.com/apache/qpid-dispatch/pull/358
  
    Code merged.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[GitHub] qpid-dispatch pull request #358: DISPATCH-930 - Initial incomplete implement...

Posted by ted-ross <gi...@git.apache.org>.
Github user ted-ross closed the pull request at:

    https://github.com/apache/qpid-dispatch/pull/358


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org