tommyettinger commented on issue #2345: URL: https://github.com/apache/fory/issues/2345#issuecomment-2986298380
So the exception's text had me quite confused as to why some tests of mine were passing and some weren't. It turns out extending `MapSerializer` is sufficient, it doesn't need to be `AbstractMapSerializer`, and `MapSerializer` seems to be much easier to extend right now, at least for me. You can close this when you feel like the "mysterious NPE" is solved, which I think it might be already. I do think the exception text should mention extending "MapSerializer or AbstractMapSerializer" -- the mechanism that AbstractMapSerializer works by isn't clear to me but MapSerializer works fine. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
