[ 
https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16253517#comment-16253517
 ] 

ASF GitHub Bot commented on THRIFT-4329:
----------------------------------------

Github user jeking3 commented on the issue:

    https://github.com/apache/thrift/pull/1361
  
    I also fail locally running in the docker container (but csharp does not, 
which is interesting):
    ```
    
===============================================================================
    *** Following 4 failures were unexpected ***:
    If it is introduced by you, please fix it before submitting the code.
    
===============================================================================
    server-client:          protocol:         transport:               result:
    c_glib-rs               multi             buffered-ip              
failure(timeout)
    c_glib-rs               multi             framed-ip                
failure(timeout)
    c_glib-rs               multic            buffered-ip              
failure(timeout)
    c_glib-rs               multic            framed-ip                
failure(timeout)
    
===============================================================================
    Unexpected failures are logged to test/log/unexpected_failures.log
    You can browse results at:
        file:///thrift/src/test/index.html
    # If you use Chrome, run:
    #   cd /thrift/src
    #   python -m http.server 8001
    # then browse:
    #   http://localhost:8001/test/
    Full log for each test is here:
        test/log/server_client_protocol_transport_client.log
        test/log/server_client_protocol_transport_server.log
    4 failed of 2590 tests in total.
    Test execution took 887.2 seconds.
    Wed Nov 15 14:08:42 2017
    Makefile:1097: recipe for target 'cross-.*' failed
    make: *** [cross-.*] Error 1
    
    RET=$?
    if [ $RET -ne 0 ]; then
      cat test/log/unexpected_failures.log
    fi
    TEST: [c_glib-rs_multi_buffered-ip]
    *************************** server message ***************************
    Wed Nov 15 13:54:58 2017
    Executing: /thrift/src/test/c_glib/test_server --lt-debug --protocol=multi 
--transport=buffered --port=39553
    Directory: /thrift/src/test/c_glib
    config:delay: 5
    config:timeout: 6
    
===============================================================================
    test_server:test_server:93: libtool wrapper (GNU libtool) 2.4.6 
Debian-2.4.6-0.1
    test_server:test_server:114: newargv[0]: 
/thrift/src/test/c_glib/.libs/lt-test_server
    test_server:test_server:104: newargv[1]: --protocol=multi
    test_server:test_server:104: newargv[2]: --transport=buffered
    test_server:test_server:104: newargv[3]: --port=39553
    Starting "simple" server (buffered/binary:multi) listen on: 39553
    
    (lt-test_server:36882): GLib-WARNING **: GError set over the top of a 
previous GError or uninitialized memory.
    This indicates a bug in someone's code. You must ensure an error is NULL 
before it's set.
    The overwriting error message was: message type invalid for this processor
    
    (lt-test_server:36882): GLib-WARNING **: GError set over the top of a 
previous GError or uninitialized memory.
    This indicates a bug in someone's code. You must ensure an error is NULL 
before it's set.
    The overwriting error message was: failed to read 1 bytes - Success
    ** Message: thrift_simple_server_serve: failed to read 4 bytes - Success
    
    Server process is successfully killed.
    
===============================================================================
    Process is killed.
    Test execution took 6.2 seconds.
    Wed Nov 15 13:55:04 2017
    
    *************************** client message ***************************
    Wed Nov 15 13:54:58 2017
    Executing: /thrift/src/test/rs/bin/test_client --protocol=multi 
--transport=buffered --port=39553
    Directory: /thrift/src/test/rs/bin
    config:delay: 5
    config:timeout: 6
    
===============================================================================
    INFO:test_client: connecting to 127.0.0.1:39553 with multi+buffered stack
    INFO:test_client: testVoid
    INFO:test_client: testString
    INFO:test_client: testBool
    INFO:test_client: testBool
    INFO:test_client: testByte
    INFO:test_client: testi32
    INFO:test_client: testi64
    INFO:test_client: testDouble
    INFO:test_client: testTypedef
    INFO:test_client: testEnum
    INFO:test_client: testBinary
    INFO:test_client: testStruct
    INFO:test_client: testNest
    INFO:test_client: SecondService blahBlah
    
===============================================================================
    Return code: -9
    Test execution took 6.0 seconds.
    Wed Nov 15 13:55:04 2017
    
    **********************************************************************
    
    TEST: [c_glib-rs_multi_framed-ip]
    *************************** server message ***************************
    Wed Nov 15 13:54:58 2017
    Executing: /thrift/src/test/c_glib/test_server --lt-debug --protocol=multi 
--transport=framed --port=44309
    Directory: /thrift/src/test/c_glib
    config:delay: 5
    config:timeout: 6
    
===============================================================================
    test_server:test_server:93: libtool wrapper (GNU libtool) 2.4.6 
Debian-2.4.6-0.1
    test_server:test_server:114: newargv[0]: 
/thrift/src/test/c_glib/.libs/lt-test_server
    test_server:test_server:104: newargv[1]: --protocol=multi
    test_server:test_server:104: newargv[2]: --transport=framed
    test_server:test_server:104: newargv[3]: --port=44309
    Starting "simple" server (framed/binary:multi) listen on: 44309
    
    (lt-test_server:36899): GLib-WARNING **: GError set over the top of a 
previous GError or uninitialized memory.
    This indicates a bug in someone's code. You must ensure an error is NULL 
before it's set.
    The overwriting error message was: message type invalid for this processor
    
    (lt-test_server:36899): GLib-WARNING **: GError set over the top of a 
previous GError or uninitialized memory.
    This indicates a bug in someone's code. You must ensure an error is NULL 
before it's set.
    The overwriting error message was: failed to read 4 bytes - Success
    ** Message: thrift_simple_server_serve: failed to read 4 bytes - Success
    
    Server process is successfully killed.
    
===============================================================================
    Process is killed.
    Test execution took 6.2 seconds.
    Wed Nov 15 13:55:04 2017
    
    *************************** client message ***************************
    Wed Nov 15 13:54:58 2017
    Executing: /thrift/src/test/rs/bin/test_client --protocol=multi 
--transport=framed --port=44309
    Directory: /thrift/src/test/rs/bin
    config:delay: 5
    config:timeout: 6
    
===============================================================================
    INFO:test_client: connecting to 127.0.0.1:44309 with multi+framed stack
    INFO:test_client: testVoid
    INFO:test_client: testString
    INFO:test_client: testBool
    INFO:test_client: testBool
    INFO:test_client: testByte
    INFO:test_client: testi32
    INFO:test_client: testi64
    INFO:test_client: testDouble
    INFO:test_client: testTypedef
    INFO:test_client: testEnum
    INFO:test_client: testBinary
    INFO:test_client: testStruct
    INFO:test_client: testNest
    INFO:test_client: SecondService blahBlah
    
===============================================================================
    Return code: -9
    Test execution took 6.0 seconds.
    Wed Nov 15 13:55:04 2017
    
    **********************************************************************
    
    TEST: [c_glib-rs_multic_buffered-ip]
    *************************** server message ***************************
    Wed Nov 15 13:55:00 2017
    Executing: /thrift/src/test/c_glib/test_server --lt-debug --protocol=multic 
--transport=buffered --port=37943
    Directory: /thrift/src/test/c_glib
    config:delay: 5
    config:timeout: 6
    
===============================================================================
    test_server:test_server:93: libtool wrapper (GNU libtool) 2.4.6 
Debian-2.4.6-0.1
    test_server:test_server:114: newargv[0]: 
/thrift/src/test/c_glib/.libs/lt-test_server
    test_server:test_server:104: newargv[1]: --protocol=multic
    test_server:test_server:104: newargv[2]: --transport=buffered
    test_server:test_server:104: newargv[3]: --port=37943
    Starting "simple" server (buffered/compact:multic) listen on: 37943
    
    (lt-test_server:36956): GLib-WARNING **: GError set over the top of a 
previous GError or uninitialized memory.
    This indicates a bug in someone's code. You must ensure an error is NULL 
before it's set.
    The overwriting error message was: message type invalid for this processor
    
    (lt-test_server:36956): GLib-WARNING **: GError set over the top of a 
previous GError or uninitialized memory.
    This indicates a bug in someone's code. You must ensure an error is NULL 
before it's set.
    The overwriting error message was: failed to read 1 bytes - Success
    ** Message: thrift_simple_server_serve: failed to read 1 bytes - Success
    
    Server process is successfully killed.
    
===============================================================================
    Process is killed.
    Test execution took 6.2 seconds.
    Wed Nov 15 13:55:07 2017
    
    *************************** client message ***************************
    Wed Nov 15 13:55:01 2017
    Executing: /thrift/src/test/rs/bin/test_client --protocol=multic 
--transport=buffered --port=37943
    Directory: /thrift/src/test/rs/bin
    config:delay: 5
    config:timeout: 6
    
===============================================================================
    INFO:test_client: connecting to 127.0.0.1:37943 with multic+buffered stack
    INFO:test_client: testVoid
    INFO:test_client: testString
    INFO:test_client: testBool
    INFO:test_client: testBool
    INFO:test_client: testByte
    INFO:test_client: testi32
    INFO:test_client: testi64
    INFO:test_client: testDouble
    INFO:test_client: testTypedef
    INFO:test_client: testEnum
    INFO:test_client: testBinary
    INFO:test_client: testStruct
    INFO:test_client: testNest
    INFO:test_client: SecondService blahBlah
    
===============================================================================
    Return code: -9
    Test execution took 6.0 seconds.
    Wed Nov 15 13:55:07 2017
    
    **********************************************************************
    
    TEST: [c_glib-rs_multic_framed-ip]
    *************************** server message ***************************
    Wed Nov 15 13:55:01 2017
    Executing: /thrift/src/test/c_glib/test_server --lt-debug --protocol=multic 
--transport=framed --port=42147
    Directory: /thrift/src/test/c_glib
    config:delay: 5
    config:timeout: 6
    
===============================================================================
    test_server:test_server:93: libtool wrapper (GNU libtool) 2.4.6 
Debian-2.4.6-0.1
    test_server:test_server:114: newargv[0]: 
/thrift/src/test/c_glib/.libs/lt-test_server
    test_server:test_server:104: newargv[1]: --protocol=multic
    test_server:test_server:104: newargv[2]: --transport=framed
    test_server:test_server:104: newargv[3]: --port=42147
    Starting "simple" server (framed/compact:multic) listen on: 42147
    
    (lt-test_server:36986): GLib-WARNING **: GError set over the top of a 
previous GError or uninitialized memory.
    This indicates a bug in someone's code. You must ensure an error is NULL 
before it's set.
    The overwriting error message was: message type invalid for this processor
    
    (lt-test_server:36986): GLib-WARNING **: GError set over the top of a 
previous GError or uninitialized memory.
    This indicates a bug in someone's code. You must ensure an error is NULL 
before it's set.
    The overwriting error message was: failed to read 4 bytes - Success
    ** Message: thrift_simple_server_serve: failed to read 4 bytes - Success
    
    Server process is successfully killed.
    
===============================================================================
    Process is killed.
    Test execution took 6.2 seconds.
    Wed Nov 15 13:55:07 2017
    
    *************************** client message ***************************
    Wed Nov 15 13:55:01 2017
    Executing: /thrift/src/test/rs/bin/test_client --protocol=multic 
--transport=framed --port=42147
    Directory: /thrift/src/test/rs/bin
    config:delay: 5
    config:timeout: 6
    
===============================================================================
    INFO:test_client: connecting to 127.0.0.1:42147 with multic+framed stack
    INFO:test_client: testVoid
    INFO:test_client: testString
    INFO:test_client: testBool
    INFO:test_client: testBool
    INFO:test_client: testByte
    INFO:test_client: testi32
    INFO:test_client: testi64
    INFO:test_client: testDouble
    INFO:test_client: testTypedef
    INFO:test_client: testEnum
    INFO:test_client: testBinary
    INFO:test_client: testStruct
    INFO:test_client: testNest
    INFO:test_client: SecondService blahBlah
    
===============================================================================
    Return code: -9
    Test execution took 6.0 seconds.
    Wed Nov 15 13:55:07 2017
    
    **********************************************************************
    ```


> c_glib Doesn't have a multiplexed processor
> -------------------------------------------
>
>                 Key: THRIFT-4329
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4329
>             Project: Thrift
>          Issue Type: Bug
>          Components: C glib - Library
>    Affects Versions: 0.10.0
>            Reporter: Gonzalo Aguilar
>            Assignee: Gonzalo Aguilar
>
> It seems that multiplexed protocol only implements 
> thrift_multiplexed_protocol_write_message_begin that's ok for sending 
> messages to a multiplexed server but not for the C server. We also need a 
> multiplexed processor for the server.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to