thrift-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (THRIFT-3380) nodejs: 0.9.2 -> 0.9.3 upgrade breaks Protocol and Transport requires
Date Sun, 02 Apr 2017 05:43:42 GMT


ASF GitHub Bot commented on THRIFT-3380:

Github user jeking3 commented on the issue:
    @jfarrell this was marked Won't Fix in Jira.  This PR can be closed.

> nodejs: 0.9.2 -> 0.9.3 upgrade breaks Protocol and Transport requires
> ---------------------------------------------------------------------
>                 Key: THRIFT-3380
>                 URL:
>             Project: Thrift
>          Issue Type: Bug
>          Components: Node.js - Library
>    Affects Versions: 0.9.3
>            Reporter: Matt Willer
>            Assignee: Roger Meier
>            Priority: Critical
>             Fix For: 0.10.0
>   Original Estimate: 10m
>  Remaining Estimate: 10m
> Node.js projects that depend on Thrift and need to use a specific transport or protocol
must require them in separately, like this:
> {code}
> var thrift = require('thrift'),
>     ThriftTransports = require('thrift/lib/thrift/transport');
> {code}
> The new version (0.9.3) changed that directory structure of the thrift module so that
the transport file is now located at thrift/lib/nodejs/lib/thrift/transport.js, which breaks
any application that was requiring it at the old path.  This type of breaking change is inappropriate
for a patch version, and should be fixed immediately.
> The directory structure change also has the undesirable side effect of including every
single language implementation of thrift in the Node.js module, bloating the size of the module
with unnecessary files.
> Long-term, the right fix for this is to export useful parts of the library (e.g. transport
and protocol constructors) from the main file, but since this is a patch version the immediate
fix should be to maintain existing behavior.

This message was sent by Atlassian JIRA

View raw message