accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From keith-turner <...@git.apache.org>
Subject [GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for RFile
Date Tue, 31 May 2016 14:57:32 GMT
Github user keith-turner commented on the pull request:

    https://github.com/apache/accumulo/pull/103
  
    >  Those who wish to implement the layer of access you've allowed will do so without
regard to an external API. 
    
    Using non public Accumulo APIs is ok if the software is only used by the authors.  However
in the case where someone writes software that uses non-public APIs and shares that software
with others, it could result in a bad/confusing user experience.
    
    > What we needed and implemented was a way to bypass the tserver. I would like to see
this supported through the connector as an established mechanism for data retrieval. 
    
    What I was going for here is a different use case.  This API supports working with RFiles
without having an Accumulo instance.   I made a choice not to require a connector.  This supports
writing tools  that can be shared around reading and writing rfiles.
    
    > We want to hide all of this. I would love to see all of that implemented for us so
we don't need to continue to implement what we currently have. I was hoping your pull requests
would allow us to that. Perhaps i'm reaching for what I hoped the intent of your pull request
was. 
    
    So I think this puts more in the public API to support this use case, but not everything.
 We do need to work towards getting this use case fully supported in the public API.  Is it
documented in an issue?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message