[ 
https://issues.apache.org/jira/browse/THRIFT-3783?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15230860#comment-15230860
 ] 

ASF GitHub Bot commented on THRIFT-3783:
----------------------------------------

Github user Jens-G commented on a diff in the pull request:

    https://github.com/apache/thrift/pull/982#discussion_r58928719
  
    --- Diff: compiler/cpp/src/generate/t_py_generator.cc ---
    @@ -21,12 +21,19 @@
     #include <fstream>
     #include <iostream>
     #include <vector>
    +#include <list>
    +#include <map>
     
     #include <stdlib.h>
     #include <sys/stat.h>
     #include <sys/types.h>
     #include <sstream>
     #include <algorithm>
    +#include <boost/graph/adjacency_list.hpp>
    +#include <boost/graph/topological_sort.hpp>
    +#include <boost/graph/depth_first_search.hpp>
    +#include <boost/graph/visitors.hpp>
    --- End diff --
    
    We don't want to add boost as a dependency when building the Thrift 
compiler. Sorry.


> python code generator dose not handle struct dependent
> ------------------------------------------------------
>
>                 Key: THRIFT-3783
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3783
>             Project: Thrift
>          Issue Type: Bug
>          Components: Python - Compiler
>            Reporter: Huabin
>            Priority: Minor
>
> given thrift idl
> {code}
> struct A {
> 1: B b,
> }
> struct B {
> 1: i32 b,
> }
> {code}
> generated ttypes.py
> {code}
> 20   class A:
>  21   """
>  22   Attributes:
>  23    - b
>  24   """
>  25  
>  26   thrift_spec = (
>  27     None, # 0
>  28     (1, TType.STRUCT, 'b', (B, B.thrift_spec), None, ), # 1
>  29   )
>  30  
> {code}
> import A will cause error since it referenced B, which has not be defined.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to