[GitHub] qpid-dispatch pull request #423: DISPATCH-1194: Link Route address lookup fo...

2018-12-03 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/423 DISPATCH-1194: Link Route address lookup for edge router You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch DISPATCH-1194

[GitHub] qpid-dispatch pull request #422: DISPATCH-1205: accept valid integer sign co...

2018-12-03 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/422 DISPATCH-1205: accept valid integer sign coversions when parsing You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch

[GitHub] qpid-dispatch pull request #417: NO-JIRA: Add a tool for post-processing val...

2018-11-09 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/417 NO-JIRA: Add a tool for post-processing valgrind test results You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch grinder

[GitHub] qpid-dispatch pull request #411: DISPATCH-1154: Synchronize edge link routes...

2018-10-31 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/411 DISPATCH-1154: Synchronize edge link routes to interior You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch DISPATCH-1154

[GitHub] qpid-dispatch pull request #394: DISPATCH-1150: Router core messaging client

2018-10-23 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/394 DISPATCH-1150: Router core messaging client You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch DISPATCH-1150

[GitHub] qpid-dispatch pull request #393: DISPATCH-1143: New feature: connection-scop...

2018-10-19 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/393#discussion_r226763750 --- Diff: python/qpid_dispatch_internal/management/agent.py --- @@ -417,6 +417,13 @@ def create(self): def __str__(self): return

[GitHub] qpid-dispatch pull request #393: DISPATCH-1143: New feature: connection-scop...

2018-10-19 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/393#discussion_r226743048 --- Diff: include/qpid/dispatch/router_core.h --- @@ -721,10 +723,11 @@ typedef struct qdr_query_t qdr_query_t; * @param name The name supplied

[GitHub] qpid-dispatch pull request #393: DISPATCH-1143: New feature: connection-scop...

2018-10-19 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/393#discussion_r226742384 --- Diff: python/qpid_dispatch_internal/management/config.py --- @@ -67,6 +67,7 @@ def transform_sections(sections): if s[0

[GitHub] qpid-dispatch pull request #393: DISPATCH-1143: New feature: connection-scop...

2018-10-19 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/393#discussion_r226742200 --- Diff: python/qpid_dispatch_internal/management/agent.py --- @@ -417,6 +417,13 @@ def create(self): def __str__(self): return

[GitHub] qpid-dispatch pull request #393: DISPATCH-1143: New feature: connection-scop...

2018-10-19 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/393 DISPATCH-1143: New feature: connection-scoped link routes Haven't changed the name yet - still attachSubscription. Will do that with a separate patch once the link route replication stuff

[GitHub] qpid-dispatch pull request #388: DISPATCH-1131: add better validation of add...

2018-10-09 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/388 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #388: DISPATCH-1131: add better validation of add...

2018-10-08 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/388 DISPATCH-1131: add better validation of address patterns and prefixes This moves the address pattern/prefix validation to a common function that can be share by both link routes add

[GitHub] qpid-dispatch pull request #387: DISPATCH-1131: add better validation of add...

2018-10-08 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/387 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #387: DISPATCH-1131: add better validation of add...

2018-10-04 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/387 DISPATCH-1131: add better validation of address patterns and prefixes This moves the address pattern/prefix validation to a common function that can be share by both link routes add

[GitHub] qpid-dispatch issue #385: DISPATCH-1131: add better validation of address pa...

2018-10-04 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-dispatch/pull/385 Oops left in an extraneous check.. --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands

[GitHub] qpid-dispatch pull request #385: DISPATCH-1131: add better validation of add...

2018-10-04 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/385 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #386: DISPATCH-1137: fix stale container name poi...

2018-10-03 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/386 DISPATCH-1137: fix stale container name pointer in connection info You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch

[GitHub] qpid-dispatch pull request #385: DISPATCH-1131: add better validation of add...

2018-09-27 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/385 DISPATCH-1131: add better validation of address patterns and prefixes This moves the address pattern/prefix validation to a common function that can be share by both link routes add

[GitHub] qpid-dispatch pull request #381: Tross dispatch 1123 1 dlv

2018-09-20 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/381 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #381: Tross dispatch 1123 1 dlv

2018-09-20 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/381 Tross dispatch 1123 1 dlv You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch tross-DISPATCH-1123-1-dlv Alternatively you

[GitHub] qpid-dispatch pull request #378: DISPATCH-1121: Set the parse error flag on ...

2018-09-15 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/378 DISPATCH-1121: Set the parse error flag on 32bit under/overflow You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch

[GitHub] qpid-dispatch pull request #377: DISPATCH-930: template code for edge route ...

2018-09-13 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/377 DISPATCH-930: template code for edge route connection events You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch DISPATCH

[GitHub] qpid-dispatch pull request #359: DISPATCH-1100: add auto delete linkRoutes

2018-08-24 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/359#discussion_r212738886 --- Diff: src/router_core/route_control.c --- @@ -189,37 +189,40 @@ static void qdr_link_route_activate_CT(qdr_core_t *core, qdr_link_route_t *lr, q

[GitHub] qpid-dispatch pull request #361: DISPATCH-1102: release prefix strings in qd...

2018-08-10 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/361 DISPATCH-1102: release prefix strings in qdr_address_t cleanup You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch DISPATCH

[GitHub] qpid-dispatch pull request #360: NO-JIRA: fix build when -DUSE_MEMORY_POOL=F...

2018-08-10 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/360 NO-JIRA: fix build when -DUSE_MEMORY_POOL=False You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch IOVEC_FIX

[GitHub] qpid-dispatch pull request #359: DISPATCH-1100: add auto delete linkRoutes

2018-08-10 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/359 DISPATCH-1100: add auto delete linkRoutes You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch DISPATCH-1100 Alternatively

[GitHub] qpid-dispatch pull request #349: DISPATCH-1092: clear the cached session poi...

2018-08-01 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/349 DISPATCH-1092: clear the cached session pointer when the session closes You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch

[GitHub] qpid-dispatch pull request #335: Dispatch 1046 rev2

2018-06-28 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/335 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #335: Dispatch 1046 rev2

2018-06-27 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/335 Dispatch 1046 rev2 Had to change the way we pass args to the host address parse tree api under python3 c_char_p is a bytes type under python3. Passing and returning a unicode

[GitHub] qpid-dispatch pull request #330: DISPATCH-1048: correct import for urllib in...

2018-06-23 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/330 DISPATCH-1048: correct import for urllib in python3, use unicode You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch

[GitHub] qpid-dispatch pull request #329: DISPATCH-1047: open the pipe to saslpasswd2...

2018-06-23 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/329 DISPATCH-1047: open the pipe to saslpasswd2 in text mode You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch DISPATCH-1047

[GitHub] qpid-proton issue #149: Do not merge - just testing....

2018-06-13 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-proton/pull/149 Done testing --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h

[GitHub] qpid-proton pull request #149: Do not merge - just testing....

2018-06-13 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-proton/pull/149 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-proton pull request #149: Do not merge - just testing....

2018-06-13 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-proton/pull/149 Do not merge - just testing Sorry for the noise, please disregard. I will delete this PR once travis is done. You can merge this pull request into a Git repository by running: $ git

[GitHub] qpid-dispatch pull request #321: NO-JIRA: enable code coverage report in tra...

2018-06-12 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/321 NO-JIRA: enable code coverage report in travis This patch enables code coverage reporting on travis. If the build + tests are successful, at the end of the output log a url

[GitHub] qpid-dispatch pull request #305: Dispatch 965 - Port to Python 3.x

2018-05-16 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/305#discussion_r188773750 --- Diff: console/config/mock/section.py --- @@ -17,6 +17,11 @@ # under the License. # +from __future__ import print_function

[GitHub] qpid-dispatch pull request #305: Dispatch 965 - Port to Python 3.x

2018-05-16 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/305 Dispatch 965 - Port to Python 3.x This patch is about as done as I can get it. Unit tests pass on 2.6, 2.7, and 3.6. Other than the fixups there are 2 major changes here

[GitHub] qpid-dispatch pull request #297: v1.1.x coverity fix by gmurthy

2018-05-01 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/297 v1.1.x coverity fix by gmurthy You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch gmurthy-hack Alternatively you can

[GitHub] qpid-dispatch pull request #272: DISPATCH-947: fix system_tests_two_routers....

2018-03-27 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/272 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #272: DISPATCH-947: fix system_tests_two_routers....

2018-03-27 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/272 DISPATCH-947: fix system_tests_two_routers.test_17_address_wildcard You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-03-22 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/244 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch issue #244: DISPATCH-89: Exchange Binding forwarder (experimen...

2018-03-22 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-dispatch/pull/244 Merged https://issues.apache.org/jira/browse/DISPATCH-89 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org

[GitHub] qpid-dispatch pull request #257: DISPATCH-914: correctly free mutex from qd_...

2018-02-23 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/257 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #258: DISPATCH-889: fix the parse tree token stri...

2018-02-23 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/258 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #259: DISPATCH-882: delay settlement until after ...

2018-02-23 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/259 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #260: DISPATCH-931: update dockerfiles with new d...

2018-02-22 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/260 DISPATCH-931: update dockerfiles with new dependency and fix syntax e… …rror You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti

[GitHub] qpid-dispatch pull request #259: DISPATCH-882: delay settlement until after ...

2018-02-15 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/259 DISPATCH-882: delay settlement until after the i/o thread puts the de… …livery on the proper list (cherry picked from commit 0118660ca013d3f524cad7bb1978d92c17bbe6eb) You can

[GitHub] qpid-dispatch pull request #258: DISPATCH-889: fix the parse tree token stri...

2018-02-15 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/258 DISPATCH-889: fix the parse tree token string comparison Backported from master You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti

[GitHub] qpid-dispatch pull request #257: DISPATCH-914: correctly free mutex from qd_...

2018-02-15 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/257 DISPATCH-914: correctly free mutex from qd_connector_t Backport for 1.0.1 release You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti

[GitHub] qpid-dispatch pull request #250: DISPATCH-919: if python-unittest2 not insta...

2018-01-30 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/250 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch issue #249: DISPATCH-921 Run npm install after stand-alone con...

2018-01-29 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-dispatch/pull/249 Other than the version issue this patch appears to work for me! --- - To unsubscribe, e-mail: dev-unsubscr

[GitHub] qpid-dispatch pull request #249: DISPATCH-921 Run npm install after stand-al...

2018-01-29 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/249#discussion_r164544618 --- Diff: console/CMakeLists.txt --- @@ -25,33 +25,46 @@ set(CONSOLE_BASE_SOURCE_DIR "${CMAKE_SOURCE_DIR}/console/stand-alone/")

[GitHub] qpid-dispatch pull request #250: DISPATCH-919: if python-unittest2 not insta...

2018-01-29 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/250 DISPATCH-919: if python-unittest2 not installed issue warning and dis… …able tests You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-01-24 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/244#discussion_r163577138 --- Diff: python/qpid_dispatch/management/qdrouter.json --- @@ -1126,6 +1126,106

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-01-23 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/244#discussion_r163335294 --- Diff: python/qpid_dispatch/management/qdrouter.json --- @@ -1126,6 +1126,106

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-01-23 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/244#discussion_r163282364 --- Diff: src/iterator.c --- @@ -909,3 +913,40 @@ void qd_iterator_get_view_cursor( ptr->cursor= iter->view_pointer.cursor;

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-01-23 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/244#discussion_r163280167 --- Diff: python/qpid_dispatch/management/qdrouter.json --- @@ -1126,6 +1126,106

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-01-23 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/244#discussion_r163279263 --- Diff: python/qpid_dispatch/management/qdrouter.json --- @@ -1126,6 +1126,106

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-01-23 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/244#discussion_r163277543 --- Diff: python/qpid_dispatch/management/qdrouter.json --- @@ -1126,6 +1126,106

[GitHub] qpid-dispatch issue #246: DISPATCH-916 - Honor the type argument to qdmanage...

2018-01-23 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-dispatch/pull/246 Works for me. --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-01-16 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/244#discussion_r161905217 --- Diff: tests/system_tests_exchange_bindings.py --- @@ -0,0 +1,676 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-01-16 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/244#discussion_r161904967 --- Diff: src/router_core/exchange_bindings.c --- @@ -0,0 +1,1336 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] qpid-dispatch pull request #244: DISPATCH-89: Exchange Binding forwarder (ex...

2018-01-16 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/244 DISPATCH-89: Exchange Binding forwarder (experimental) Adds a forwarder that models the pre AMQP 1.0 model of exchanges and bindings. Exchanges can be added via management and bound

[GitHub] qpid-dispatch pull request #243: DISPATCH-914: correctly free mutex from qd_...

2018-01-16 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/243 DISPATCH-914: correctly free mutex from qd_connector_t You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch DISPATCH-914

[GitHub] qpid-proton pull request #134: PROTON-1730: add python version 3.6 to tox te...

2018-01-12 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-proton/pull/134 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-proton issue #134: PROTON-1730: add python version 3.6 to tox tests

2018-01-12 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-proton/pull/134 Merged --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h

[GitHub] qpid-proton pull request #134: PROTON-1730: add python version 3.6 to tox te...

2018-01-02 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-proton/pull/134 PROTON-1730: add python version 3.6 to tox tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/qpid-proton PROTON-1730

[GitHub] qpid-dispatch pull request #234: DISPATCH-89: Parse tree updates

2017-12-13 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/234 DISPATCH-89: Parse tree updates Modification of the parse tree to support MQTT and AMQP 0-10 wildcard patterns. Fairly stand alone chunk of work. You can merge this pull request

[GitHub] qpid-proton pull request #118: PROTON-1326: update python setup.py to avoid ...

2017-12-06 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-proton/pull/118 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #230: DISPATCH-892: add test suite code coverage ...

2017-12-05 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/230 DISPATCH-892: add test suite code coverage analysis This feature was copied unabashedly from the coverage work done on the Proton project by Andrew Stitcher, Jiri Danek, Chuck Rolke, et

[GitHub] qpid-dispatch pull request #229: DISPATCH-892: add test suite code coverage ...

2017-12-05 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/229 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #229: DISPATCH-892: add test suite code coverage ...

2017-12-05 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/229 DISPATCH-892: add test suite code coverage analysis This feature was copied unabashedly from the coverage work done on the Proton project by Andrew Stitcher, Jiri Danek, Chuck Rolke, et

[GitHub] qpid-dispatch pull request #225: DISPATCH-882: delay settlement until after ...

2017-11-22 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/225 DISPATCH-882: delay settlement until after the i/o thread puts the de… …livery on the proper list You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] qpid-dispatch pull request #222: Add missing delivery decref when presettled...

2017-11-17 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/222 Add missing delivery decref when presettled. Related to #213 You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch decref

[GitHub] qpid-dispatch pull request #218: DISPATCH-875 - Doc new "pattern" attribute ...

2017-11-17 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/218#discussion_r151715642 --- Diff: doc/new-book/routing.adoc --- @@ -213,15 +213,17 @@ address { } -`prefix` | `pattern`:: The address or group

[GitHub] qpid-dispatch pull request #218: DISPATCH-875 - Doc new "pattern" attribute ...

2017-11-16 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/218#discussion_r151509115 --- Diff: doc/new-book/routing.adoc --- @@ -213,15 +213,17 @@ address { } -`prefix` | `pattern`:: The address or group

[GitHub] qpid-dispatch pull request #218: DISPATCH-875 - Doc new "pattern" attribute ...

2017-11-16 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/218#discussion_r151441999 --- Diff: doc/new-book/routing.adoc --- @@ -213,15 +213,17 @@ address { } -`prefix` | `pattern`:: The address or group

[GitHub] qpid-dispatch pull request #218: DISPATCH-875 - Doc new "pattern" attribute ...

2017-11-15 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/218#discussion_r151167247 --- Diff: doc/new-book/routing.adoc --- @@ -288,11 +307,13 @@ address { } -`prefix`:: The address prefix that matches

[GitHub] qpid-dispatch pull request #218: DISPATCH-875 - Doc new "pattern" attribute ...

2017-11-15 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/218#discussion_r151166644 --- Diff: doc/new-book/routing.adoc --- @@ -213,11 +213,30 @@ address { } -`prefix`:: The address prefix. All messages

[GitHub] qpid-dispatch pull request #218: DISPATCH-875 - Doc new "pattern" attribute ...

2017-11-15 Thread kgiusti
Github user kgiusti commented on a diff in the pull request: https://github.com/apache/qpid-dispatch/pull/218#discussion_r151165869 --- Diff: doc/new-book/routing.adoc --- @@ -213,11 +213,30 @@ address { } -`prefix`:: The address prefix. All messages

[GitHub] qpid-dispatch pull request #139: [WIP] Exchange Binding Forwarder

2017-10-30 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-dispatch/pull/139 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-cpp pull request #4: QPID-7702: Fix some minor memory leaks detected by...

2017-09-29 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-cpp/pull/4 --- - To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch pull request #202: DISPATCH-813: change lifecycle of link rout...

2017-09-27 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/202 DISPATCH-813: change lifecycle of link route to remove reference count Remove the ref counting when adding/removing link route patterns and instead simply add the pattern when the address

[GitHub] qpid-dispatch pull request #201: DISPATCH-813: add address pattern matching ...

2017-09-22 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/201 DISPATCH-813: add address pattern matching to link routes You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti/dispatch DISPATCH-813

[GitHub] qpid-dispatch pull request #197: DISPATCH-821: actually delete the configure...

2017-09-15 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/197 DISPATCH-821: actually delete the configured address Well. that was stupid. You can merge this pull request into a Git repository by running: $ git pull https://github.com/kgiusti

[GitHub] qpid-proton issue #118: PROTON-1326: update python setup.py to avoid using i...

2017-08-25 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-proton/pull/118 The failures don't seem to be related - various test failures but not the python-tox tests (which rely on setup.py patched by this patch). Yes - I would have to create a 0.17.x branch

[GitHub] qpid-proton pull request #118: PROTON-1326: update python setup.py to avoid ...

2017-08-25 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-proton/pull/118 PROTON-1326: update python setup.py to avoid using incompatible opens… …sl 1.1.0 Proposing this patch against 0.17.x to fix a problem in the python-qpid-proton pypi packages

[GitHub] qpid-dispatch pull request #181: Dispatch 731- configure wildcard addresses

2017-07-25 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/181 Dispatch 731- configure wildcard addresses Passes existing tests. Todo: 1) unit tests 2) cleanup lots of iterator->string->iterator transformations 3) doc

[GitHub] qpid-proton pull request #102: NO-JIRA: revert temporary workaround for PROT...

2017-04-13 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-proton/pull/102 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] qpid-proton issue #100: PROTON-1408 -- fixes for message ID wraparound bugs

2017-03-31 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-proton/pull/100 Fyi - I've checked in a fix to the travis issue. Rebase to master and re-run the travis tests - they should pass. --- If your project is set up for it, you can reply to this email and have

[GitHub] qpid-proton pull request #102: NO-JIRA: revert temporary workaround for PROT...

2017-03-31 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-proton/pull/102 NO-JIRA: revert temporary workaround for PROTON-1453 Looks like the fix for PROTON-1453 resolves the problems travis hit for the tox tests. This change reverts just the change where

[GitHub] qpid-proton issue #101: PROTON-1453: properly terminate output string buffer

2017-03-31 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-proton/pull/101 I agree with Alan on this - but in this specific case proton is actually *inconsistent* in the handling of this buffer. In the schannel impl the buffer is null terminated regardless of outcome

[GitHub] qpid-proton pull request #101: PROTON-1453: properly terminate output string...

2017-03-30 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-proton/pull/101 PROTON-1453: properly terminate output string buffer Fixes the occasional segfault when running travis (as encountered by aconway) You can merge this pull request into a Git repository

[GitHub] qpid-cpp pull request #4: QPID-7702: Fix some minor memory leaks detected by...

2017-03-20 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-cpp/pull/4 QPID-7702: Fix some minor memory leaks detected by Coverity scan Use SWIG_fail instead of returning on error. Avoid possible leaks during exception handling. You can merge this pull request

[GitHub] qpid-cpp issue #3: QPID-7702: use SWIG_fail instead of returning on error

2017-03-20 Thread kgiusti
Github user kgiusti commented on the issue: https://github.com/apache/qpid-cpp/pull/3 Have an updated patch coming... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] qpid-cpp pull request #3: QPID-7702: use SWIG_fail instead of returning on e...

2017-03-20 Thread kgiusti
Github user kgiusti closed the pull request at: https://github.com/apache/qpid-cpp/pull/3 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] qpid-proton pull request #98: PROTON-1434: add test blacklist to tox tests

2017-03-13 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-proton/pull/98 PROTON-1434: add test blacklist to tox tests Helps speed up the time spent running the tox tests You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] qpid-cpp pull request #3: QPID-7702: use SWIG_fail instead of returning on e...

2017-03-13 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-cpp/pull/3 QPID-7702: use SWIG_fail instead of returning on error This change should prevent many of the wrapper-related resource leaks found by Coverity. You can merge this pull request into a Git

[GitHub] qpid-proton pull request #97: PROTON-1345: update setup.py.in with libuv and...

2017-03-07 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-proton/pull/97 PROTON-1345: update setup.py.in with libuv and cmake source list Had to update the setup.py.in file with Alan Conway's libuv/proactor stuff. You can merge this pull request into a Git

[GitHub] qpid-dispatch pull request #139: [WIP] Exchange Binding Forwarder

2017-01-20 Thread kgiusti
GitHub user kgiusti opened a pull request: https://github.com/apache/qpid-dispatch/pull/139 [WIP] Exchange Binding Forwarder Do not merge! This is a work in progress. This pull request is merely a request for feedback on the new code. You can merge this pull request

  1   2   >