You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@dubbo.apache.org by "Moriadry (GitHub)" <gi...@apache.org> on 2019/08/19 09:54:56 UTC

[GitHub] [dubbo] Moriadry commented on issue #4862: [Dubbo-4861] WIP: fix stackoverflow of protostuff and other errors

@beiwei30 I caught stackoverflow error when I add two unit tests. And it turns out the Inner class need a `static` declaration , otherwise It will show us SOF error. The current implement of Protostuff project is correct.

Do you think we need to add those two tests in `ProtostuffObjectOutputTest.java`?  If not so, I will close pr and issue.

[ Full content available at: https://github.com/apache/dubbo/pull/4862 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org