On 03/09/18 03:32, Jun Liu wrote:

<snip/>

> Please vote accordingly:
> [X] +1 approve 
> [ ] +0 no opinion 
> [ ] -1 disapprove with the reason

Notes:
======

Hash and signature match.

Source zip matches Git tag (apart from expected differences of
.gitignore and Maven wrapper)

Builds and all tests except one pass.


Issues from previous RC still present in this RC:
=================================================

The sha512 hashes are missing the '*' marker that indicates they are
hashes for binary files rather than text files. Trivial issue. New RC
not required.


New issues
==========

Tests seem to expect 127.0.0.2 to be a valid IP. If this is the case,
consider documenting the requirements to run the tests somewhere obvious
in the source tree. Trivial issue. New RC not required.

I see the following test failures:
  Oracle Java 8 update 181
  Ubuntu 18.04.1 LTS (fully patched)
  Maven 3.5.4

This fails consistently for me:
-------------------------------------------------------------------------------
Test set: com.alibaba.dubbo.config.AbstractInterfaceConfigTest
-------------------------------------------------------------------------------
Tests run: 38, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.119
sec <<< FAILURE! - in com.alibaba.dubbo.config.AbstractInterfaceConfigTest
checkApplication1(com.alibaba.dubbo.config.AbstractInterfaceConfigTest)
Time elapsed: 0.006 sec  <<< FAILURE!
junit.framework.ComparisonFailure: expected:<10[0]> but was:<10[]>
        at
com.alibaba.dubbo.config.AbstractInterfaceConfigTest.checkApplication1(AbstractInterfaceConfigTest.java:90)

I note that this test has been observed to fail for other community
members in earlier RCs.

I can recreate this failure on the command line but not in an IDE.

Whether this failure is significant enough to halt the release is
something for those more knowledgeable about Dubbo than I to decide.


Mark

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to