Finagle: vinculação de versão libthrift.

Criado em 6 fev. 2013  ·  13Comentários  ·  Fonte: twitter/finagle

Oi, pelo que entendi, uma vez que vocês estão se estendendo sobre algumas das APIs do thrift geradas pelo scrooge, você provavelmente está vinculado a alguma versão do thrift?

Needs Internal Work

Todos 13 comentários

Vejo a versão 0.5.0 para libthrift no pom. Isso parece muito antigo. É seguro atualizar?

158 uau, bagunçado.

Então, por que a economia de dinheiro está usando uma versão tão antiga de economia?

Estamos presos no 0.5.0 internamente no Twitter e provavelmente não iremos nos mover tão cedo.

Existem atualizações? A última atualização foi há quase um ano. Obrigado!

Olá @jduan , não é uma atualização muito boa aqui. Ainda estamos no branch 0.5.x internamente. Seria uma tarefa muito grande migrar todos internamente. Talvez algum dia, mas não estamos trabalhando ativamente nisso no momento.

@ryanking, obrigado pela atualização de qualquer maneira!

Há um problema de licenciamento ao usar com.twitter.libthrift-0.5.0-7.

Este é um trabalho derivado do apache libthrift, portanto, se enquadra no ASLv2. A cláusula 4 (d) declara que, ao distribuir o trabalho derivado, você também deve distribuir o AVISO do trabalho original. A libthrift do Twitter não faz isso, o que a coloca em violação da licença.

Além do mais, qualquer um que puxar o finagle e redistribuir na forma binária com dependências também estará violando (o que é o meu problema). A fonte libthrift do twitter pode ser disponibilizada, ou pelo menos o AVISO adicionado ao jar.

@ivankelly , estamos planejando migrar para a versão 0.10.0 em breve, então não precisaremos mais estar em nossa bifurcação. Espero que seja em menos de um mês.

Enquanto isso, acredito que você possa editar o JAR da libthrift do seu lado para adicionar o arquivo AVISO. Alternativamente, você pode excluir nosso fork da libthrift e depender explicitamente da vanilla libthrift-0.5.0.

Desculpe pelo descuido e obrigado por nos apontar isso.

Eu acredito que você pode editar o JAR libthrift em sua extremidade para adicionar o arquivo AVISO.

Eu não tenho o arquivo de aviso, no entanto. A libthrift modificada contém alterações que são presumivelmente protegidas por direitos autorais do Twitter, portanto, é o Twitter que deve fornecer o AVISO.

Alternativamente, você pode excluir nosso fork da libthrift e depender explicitamente da vanilla libthrift-0.5.0.

Porém, não existe algum bug de simultaneidade, que foi todo o motivo do fork?

Desculpe pelo descuido e obrigado por nos apontar isso.

Sem problemas. Essas coisas são muito fáceis de ignorar.

@ivankelly não é o AVISO de que precisa de https://github.com/apache/thrift/blob/master/NOTICE ?

Quando voce diz

você distribui o trabalho derivado, você também deve distribuir o AVISO do trabalho original

Eu interpreto isso como, quando você distribui o JAR do twitter, você precisa do AVISO do apache thrift.

Se isso não funcionar para você, como você se sentiria em esperar por ~ um mês para resolvermos as coisas do 0.10.0? Está quase pronto, mas queremos esperar para fazer a transição até depois das férias.

@mosesn se alguém colocasse o AVISO de economia diretamente no frasco, estaria atribuindo os direitos autorais das modificações feitas pelo Twitter ao ASF, o que, embora eu tenha certeza de que está bom, é um pouco estranho. De qualquer forma, isso não é grande coisa. Marcando libthrift como ASLv2 no meu projeto por enquanto, e como é um projeto ASF, será coberto pelo próprio AVISO do projeto.

De qualquer forma, como eu disse, não é grande coisa, especialmente com a atualização para 0,10. Obrigado pela resposta.

Parece bom, desculpe, não tenho uma resposta melhor para você: /. Faremos o possível para que as coisas do 0.10.0 funcionem em tempo hábil.

Fechando isso, pois atualizamos para libthrift-0.10.0 a partir de https://github.com/twitter/finagle/commit/61c7a71182866250e53f9731c46f31dde28d464c.

Esta página foi útil?
0 / 5 - 0 avaliações