search

xenon-middleware / xenon-grpc

Xenon grpc based server
Apache License 2.0
0 stars 1 forks source link

Add test for getStreams with multiple chunks #11

Closed sverhoeven closed 7 years ago

sverhoeven commented 7 years ago

Use multiple messages to cat interactive job and fetch responses after each request.

sverhoeven commented 7 years ago

See f2da2e1fcf823d2593359840d93e72c4d74a144e

sverhoeven commented 7 years ago

Added test in https://github.com/NLeSC/xenon-grpc/blob/get-streams-cat-test/src/test/java/nl/esciencecenter/xenon/grpc/jobs/LocalJobsStreamsTest.java, but if fails to recieve the first message from the server (at line #122).

The test is modeled after the example of grpc java

Even if the timeout is increased the StreamForwarder threads don't write.

atzeus commented 7 years ago

Fixed by d015ffa5904e535c6acd09d2b5d0aa25c885b223

sverhoeven commented 7 years ago

It works for you but sometimes it doesn't work for me. It stalls midway somewhere.

Created https://github.com/NLeSC/xenon-grpc/tree/debug-getstreams-test branch to find out where it stalls.