[
https://issues.apache.org/jira/browse/THRIFT-1013?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bryan Duxbury closed THRIFT-1013.
---------------------------------
Resolution: Fixed
Fix Version/s: 0.6
Assignee: Peter Schuller
I committed the version against trunk. Thanks Peter!
Please open another issue if you think there's more left to be done.
> generated java code may have name clashes with thrift library
> -------------------------------------------------------------
>
> Key: THRIFT-1013
> URL: https://issues.apache.org/jira/browse/THRIFT-1013
> Project: Thrift
> Issue Type: Bug
> Components: Java - Compiler
> Affects Versions: 0.5
> Reporter: Peter Schuller
> Assignee: Peter Schuller
> Fix For: 0.6
>
> Attachments: thrift-0_5-libfqcn.patch, thrift-r1042609-libfqcn.patch
>
>
> We had a structure called 'State' in a thrift spec which caused the generated
> Java code to have a naming conflict; State conflicts with
> TAsynchMethodCall.State (an enum).
> Since part of the point of thrift is long-term compatibility and an evolving
> thrift spec, it's not very practical to have the potential for naming
> conflicts like this. So we want to fix it. But because it is at least
> slightly painful to do, I want to make sure people are fine with the intended
> fix.
> Proposed fix:
> (1) Remove t_java_generator::java_thrift_imports() completely.
> (2) Replace all references to imported Thrift library classes so their fully
> qualified class names.
> The generated code will be a bit less readable due to fully qualified class
> names, but the code should be correct and it should be obvious what comes
> from where (if fully qualified names matter that means a human is looking at
> the code, in which case importing * is not terribly nice anyway).
> Thoughts?
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
