[jitsi-dev] [jitsi/jitsi-videobridge] Process XMPP packets in order (#352)


#1

Updates Jicoco in order to use single thread packet processor. DO NOT MERGE
You can view, comment on, or merge this pull request online at:

  https://github.com/jitsi/jitsi-videobridge/pull/352

-- Commit Summary --

  * fix(ComponentImpl): process XMPP packets in order

-- File Changes --

    M pom.xml (2)
    M src/main/java/org/jitsi/videobridge/xmpp/ComponentImpl.java (17)

-- Patch Links --

https://github.com/jitsi/jitsi-videobridge/pull/352.patch
https://github.com/jitsi/jitsi-videobridge/pull/352.diff

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/352


#2

bgrozev commented on this pull request.

@@ -79,7 +79,7 @@

     <dependency>
       <groupId>${project.groupId}</groupId>
       <artifactId>jicoco</artifactId>
- <version>1.0-20160822.191856-34</version>
+ <version>1.0-SNAPSHOT</version>

Is this intentionally SNAPSHOT and not just the new version?

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/352#pullrequestreview-6674532


#3

paweldomas commented on this pull request.

@@ -79,7 +79,7 @@

     <dependency>
       <groupId>${project.groupId}</groupId>
       <artifactId>jicoco</artifactId>
- <version>1.0-20160822.191856-34</version>
+ <version>1.0-SNAPSHOT</version>

yes, for now, Jicoco needs to be merged first

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/352


#4

Jenkins: ok to test

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/352#issuecomment-257661425


#5

@bgrozev pushed 1 commit.

3c2360b Updates jicoco

···


You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/352/files/f52fca05cb683c04756323135aeb9528fd72badf..3c2360bb4668fc05677642a06477763082579051


#6

Merged #352.

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi-videobridge/pull/352#event-844041603