You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "James E. King, III (JIRA)" <ji...@apache.org> on 2017/02/08 22:10:42 UTC
[jira] [Created] (THRIFT-4079) Generated perl code that returns
structures from included thrift files is missing a necessary use clause
James E. King, III created THRIFT-4079:
------------------------------------------
Summary: Generated perl code that returns structures from included thrift files is missing a necessary use clause
Key: THRIFT-4079
URL: https://issues.apache.org/jira/browse/THRIFT-4079
Project: Thrift
Issue Type: Bug
Components: Perl - Compiler
Affects Versions: 0.10.0
Environment: Ubuntu 14.04 LTS (perl 5.18.2) with thrift 0.10.0
Reporter: James E. King, III
Assignee: James E. King, III
Priority: Critical
I made a very simple example which I will attach, however in a nutshell if I define a structure in one thrift file like this:
{{ForeignInfo.thrift:}}
{noformat}
namespace perl org.fiction.rpc
struct ForeignInfo
{
1: string someData
}
{noformat}
Then I define a service in another namespace like this:
{{SomeService.thrift:}}
{noformat}
namespace perl org.real
include "ForeignInfo.thrift"
service Company
{
ForeignInfo.ForeignInfo getForeignInfoList();
}
{noformat}
Then I compile both of them, the resulting generated perl code in {{gen-perl/org/real/Company.pm}} has the following use clauses in it:
{noformat}
require 5.6.0;
use strict;
use warnings;
use Thrift;
use org::real::Types;
{noformat}
Later on in the file we have:
{noformat}
SWITCH: for($fid)
{
/^0$/ && do{ if ($ftype == TType::STRUCT) {
$self->{success} = new org::fiction::rpc::ForeignInfo();
$xfer += $self->{success}->read($input);
} else {
$xfer += $input->skip($ftype);
}
last; };
$xfer += $input->skip($ftype);
}
{noformat}
If you put a simple wrapper around this call, the client gets an exception:
{noformat}
Undefined subroutine &org::fiction::rpc::ForeignInfo called at gen-perl/org/real/Company.pm line 98
{noformat}
Line 98 is where {{org::fiction::rpc::ForeignInfo}} is mentioned. Without a use clause for the Types defined by the include it cannot be used.
If I add this line to the generated code in Company.pm:
{noformat}
use org::fiction::rpc::Types;
{noformat}
Then everything works.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)