You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2019/09/12 18:01:42 UTC

[GitHub] [accumulo-proxy] keith-turner opened a new issue #6: Ensure proxy only uses public API before release

keith-turner opened a new issue #6: Ensure proxy only uses public API before release
URL: https://github.com/apache/accumulo-proxy/issues/6
 
 
   When the proxy was released with Accumulo it was ok if used non-public API Accumulo code because it was always released with Accumulo.  Now that it is on an independent release schedule it should ideally only uses [Accumulo's public API](https://accumulo.apache.org/api/) because this will ensure it works with any version of Accumulo after 2.0.
   
   A first step in implementing this would be to update the pom to check the proxy source code to ensure only public API is used.  For an example of how to do this see [pom.xm line 194l](https://github.com/apache/accumulo-examples/blob/007c4ca0620834dcef5debaa6eee16d132501c02/pom.xml#L194), [contrib/checkstyle.xml](https://github.com/apache/accumulo-examples/blob/007c4ca0620834dcef5debaa6eee16d132501c02/contrib/checkstyle.xml), and [contrib/import-control.xml](https://github.com/apache/accumulo-examples/blob/007c4ca0620834dcef5debaa6eee16d132501c02/contrib/import-control.xml) in accumulo examples.  As a first step we could add the checks to the pom with exceptions for any current non-api usage.  This would prevent new non-api usage from being added while we work on removing all non-api usage.
   
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services