You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "James E. King III (JIRA)" <ji...@apache.org> on 2019/01/12 00:09:00 UTC
[jira] [Commented] (THRIFT-4103) make check fails in py.tornado
intermittently on Travis CI
[ https://issues.apache.org/jira/browse/THRIFT-4103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16740894#comment-16740894 ]
James E. King III commented on THRIFT-4103:
-------------------------------------------
The test was removed here, in a code change from [~nsuke]:
https://github.com/apache/thrift/commit/66c3dbf2dec91718922e815c50a55900b78a58e1#diff-b2d5c24250948eae49233160c4538fc5L193
> make check fails in py.tornado intermittently on Travis CI
> ----------------------------------------------------------
>
> Key: THRIFT-4103
> URL: https://issues.apache.org/jira/browse/THRIFT-4103
> Project: Thrift
> Issue Type: Bug
> Components: Python - Library
> Affects Versions: 0.10.0
> Environment: Travis CI Build Job #8
> Reporter: James E. King III
> Priority: Minor
>
> In build job https://travis-ci.org/apache/thrift/jobs/204051491 there was a failure in py.twisted make check unrelated to the code that changed in node.js:
> {noformat}
> Making check in py.tornado
> make[2]: Entering directory `/thrift/src/test/py.tornado'
> ../../compiler/cpp/thrift --gen py:tornado ../ThriftTest.thrift
> [WARNING:/thrift/src/test/ThriftTest.thrift:45] No generator named 'noexist' could be found!
> [WARNING:/thrift/src/test/ThriftTest.thrift:47] cpp generator does not accept 'noexist' as sub-namespace!
> ../../compiler/cpp/thrift --gen py:tornado ../SmallTest.thrift
> [WARNING:/thrift/src/test/SmallTest.thrift:27] "senum" is deprecated and will be removed in a future compiler version. This type should be replaced with "string".
> [WARNING:/thrift/src/test/SmallTest.thrift:27] "senum" is deprecated and will be removed in a future compiler version. This type should be replaced with "string".
> ./test_suite.py
> ..ERROR:thrift.TTornado:thrift exception in handle_stream
> Traceback (most recent call last):
> File "/thrift/src/test/py.tornado/../../lib/py/build/lib.linux-x86_64-3.4/thrift/TTornado.py", line 183, in handle_stream
> yield self._processor.process(iprot, oprot)
> File "/usr/local/lib/python2.7/dist-packages/tornado/gen.py", line 1015, in run
> value = future.result()
> File "/usr/local/lib/python2.7/dist-packages/tornado/concurrent.py", line 237, in result
> raise_exc_info(self._exc_info)
> File "/usr/local/lib/python2.7/dist-packages/tornado/gen.py", line 285, in wrapper
> yielded = next(result)
> File "/thrift/src/test/py.tornado/gen-py.tornado/ThriftTest/ThriftTest.py", line 1410, in process_testException
> yield gen.maybe_future(self._handler.testException(args.arg))
> File "./test_suite.py", line 88, in testException
> raise ValueError("foo")
> ValueError: foo
> ....F...
> ======================================================================
> FAIL: test_oneway (__main__.ThriftTestCase)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
> File "/usr/local/lib/python2.7/dist-packages/tornado/testing.py", line 136, in __call__
> result = self.orig_method(*args, **kwargs)
> File "./test_suite.py", line 193, in test_oneway
> self.assertAlmostEqual(seconds, (end - start), places=3)
> AssertionError: 0 != 0.0005118846893310547 within 3 places
> ----------------------------------------------------------------------
> Ran 10 tests in 0.034s
> FAILED (failures=1)
> make[2]: *** [check] Error 1
> make[2]: Leaving directory `/thrift/src/test/py.tornado'
> make[1]: *** [check-recursive] Error 1
> make[1]: Leaving directory `/thrift/src/test'
> make: *** [check-recursive] Error 1
> {noformat}
> It looks like a test case is not accounting for floating precision properly.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)