Thrift as Public API?

[Mike Drob <ma...@cloudera.com>]

Do we intend for our thrift RPC implementation to be consumable as public
API?

This question is inspired by recent "How to get the Accumulo statistic
data" thread over on user@, where we point a user at the GetMasterStats
class.

How stable is that in theory? I understand that in practice it probably
doesn't change much, but I also am not aware that we do anything during
upgrade and compatibility testing to ensure this.

Thoughts?

Mike

Re: Thrift as Public API?

[Christopher <ct...@apache.org>]

I agree. The thrift API should not be public API (except the proxy, of course).

--
Christopher L Tubbs II
http://gravatar.com/ctubbsii


On Thu, Jun 4, 2015 at 12:48 PM, Adam Fuchs <af...@apache.org> wrote:
> IMO we do not. The thrift interface that we currently have is not easily
> consumable (you have to understand a lot about the metadata and cluster
> layout), and we don't include it in SemVer. Additional features like
> GetMasterStats should be plumbed through to the Java API if we want to
> expose them.
>
> Adam
> On Jun 4, 2015 12:31 PM, "Mike Drob" <ma...@cloudera.com> wrote:
>
>> Do we intend for our thrift RPC implementation to be consumable as public
>> API?
>>
>> This question is inspired by recent "How to get the Accumulo statistic
>> data" thread over on user@, where we point a user at the GetMasterStats
>> class.
>>
>> How stable is that in theory? I understand that in practice it probably
>> doesn't change much, but I also am not aware that we do anything during
>> upgrade and compatibility testing to ensure this.
>>
>> Thoughts?
>>
>> Mike
>>

Re: Thrift as Public API?

[Josh Elser <jo...@gmail.com>]

Agreed.

I think in the case of GetMasterStats, we don't have any other way to 
actually do it via public API now. It would be nice to have something in 
the future, but that wouldn't help in the present.

Adam Fuchs wrote:
> IMO we do not. The thrift interface that we currently have is not easily
> consumable (you have to understand a lot about the metadata and cluster
> layout), and we don't include it in SemVer. Additional features like
> GetMasterStats should be plumbed through to the Java API if we want to
> expose them.
>
> Adam
> On Jun 4, 2015 12:31 PM, "Mike Drob"<ma...@cloudera.com>  wrote:
>
>> Do we intend for our thrift RPC implementation to be consumable as public
>> API?
>>
>> This question is inspired by recent "How to get the Accumulo statistic
>> data" thread over on user@, where we point a user at the GetMasterStats
>> class.
>>
>> How stable is that in theory? I understand that in practice it probably
>> doesn't change much, but I also am not aware that we do anything during
>> upgrade and compatibility testing to ensure this.
>>
>> Thoughts?
>>
>> Mike
>>
>

Re: Thrift as Public API?

[Adam Fuchs <af...@apache.org>]

IMO we do not. The thrift interface that we currently have is not easily
consumable (you have to understand a lot about the metadata and cluster
layout), and we don't include it in SemVer. Additional features like
GetMasterStats should be plumbed through to the Java API if we want to
expose them.

Adam
On Jun 4, 2015 12:31 PM, "Mike Drob" <ma...@cloudera.com> wrote:

> Do we intend for our thrift RPC implementation to be consumable as public
> API?
>
> This question is inspired by recent "How to get the Accumulo statistic
> data" thread over on user@, where we point a user at the GetMasterStats
> class.
>
> How stable is that in theory? I understand that in practice it probably
> doesn't change much, but I also am not aware that we do anything during
> upgrade and compatibility testing to ensure this.
>
> Thoughts?
>
> Mike
>