[
https://issues.apache.org/jira/browse/THRIFT-119?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12625502#action_12625502
]
Bryan Duxbury commented on THRIFT-119:
--------------------------------------
David, I'm fine with holding off on commit until the propagation works as
expected.
Mark, I'll edit that line to make it simpler.
As far as making the default to print out all contents, I don't think that
would achieve the desired results. For instance, if you're trying to use an
assertEquals in JUnit, it's not going to call the overload, and if they don't
match, you're still going to see all the null fields. I guess part of the
problem is that I'm treating my struct like a union, but we don't have a
separate union type. If we did, this wouldn't be a point of contention, I'd
think. The other alternative would be finding a way to support compiler
annotations to structs and making this a per-struct option, which would
certainly be cool and useful for other reasons.
> Improved toString for Java Structs
> ----------------------------------
>
> Key: THRIFT-119
> URL: https://issues.apache.org/jira/browse/THRIFT-119
> Project: Thrift
> Issue Type: Sub-task
> Components: Compiler (Java)
> Reporter: Bryan Duxbury
> Assignee: Bryan Duxbury
> Attachments: thrift-119.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
