seems no way to resolve this maybe we can only log message that this schema not support highway and select rest transport automatically
wjm wjm <zzz...@gmail.com> 于2018年9月15日周六 上午10:30写道: > problem is: protobuf not allow to define List<LIst>/ List<Map> > > wjm wjm <zzz...@gmail.com> 于2018年9月15日周六 上午10:27写道: > >> it's not protoStuff problem. >> protoStuff not suport serialize/deserialize without class >> >> Willem Jiang <willem.ji...@gmail.com> 于2018年9月15日周六 上午10:18写道: >> >>> Hi Jimin, >>> The best way is we send a PR for protoStuff to provide the solution of >>> listList/listMap, but it may not meet the needs of our release >>> schedule. >>> I don't think maintain a fork version of protoStuff is good way to go. >>> If we can wrap the protoStuff and extends it ourselves, it may meet >>> the needs of our release schedule. >>> >>> Willem Jiang >>> >>> Twitter: willemjiang >>> Weibo: 姜宁willem >>> >>> On Sat, Sep 15, 2018 at 9:36 AM wjm wjm <zzz...@gmail.com> wrote: >>> > >>> > class Test { >>> > public List<List<String>> listList; >>> > public List<Map<String, String>> listMap; >>> > } >>> > >>> > the field listList/listMap is invalid in protobuf. >>> > ----------- >>> > >>> > currently we process this by protoStuff runtimeSchema, runtimeSchema >>> > generated from Test class, and runtimeSchema can support the >>> definition of >>> > listList/listMap(that's protoStuff rule, not protobuf rule) >>> > but because there are no classes in Edge service, currently we must >>> dynamic >>> > create new classes for protoStuff, that caused many problems. >>> > >>> > as we discussed before, we will not dynamic create new classes, just >>> > serialize/deserialize by proto file, and proto file not support >>> > the definition of listList/listMap >>> > in this time, we must faced the compatible problem. >>> > what's the best of our choice...... >>> >>