You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Nihal Jain (JIRA)" <ji...@apache.org> on 2019/02/12 15:07:00 UTC
[jira] [Comment Edited] (HBASE-21881) Use Forbidden API Checker to
prevent future usages of forbidden api's
[ https://issues.apache.org/jira/browse/HBASE-21881?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16766120#comment-16766120 ]
Nihal Jain edited comment on HBASE-21881 at 2/12/19 3:06 PM:
-------------------------------------------------------------
{quote} Maybe add it to {{pluginManagement}} and add a non-voting test plugin to our yetus personality.
{quote}
[~busbey] The plugin results in a compilation error if any forbidden api is used. Do we want to just disable compilation failure (not sure whether the plugin allows to do this) and add a yetus plugin which gives -1, in case a forbidden api is used?
was (Author: nihaljain.cs):
{quote} Maybe add it to {{pluginManagement}} and add a non-voting test plugin to our yetus personality.
{quote}
The plugin results in a compilation error if any forbidden api is used. Do we want to just disable compilation failure (not sure whether the plugin allows to do this) and add a yetus plugin which gives -1, in case a forbidden api is used?
> Use Forbidden API Checker to prevent future usages of forbidden api's
> ---------------------------------------------------------------------
>
> Key: HBASE-21881
> URL: https://issues.apache.org/jira/browse/HBASE-21881
> Project: HBase
> Issue Type: Improvement
> Components: build
> Reporter: Nihal Jain
> Priority: Major
>
> [Policeman's Forbidden API Checker|https://github.com/policeman-tools/forbidden-apis] provides an elegant way to fail build if any "forbidden api" is used in the code base.
> Here we will add the plugin to our pom.xml.
> We will discuss which all api's to put in forbidden list. Or whether to use the one's bundled with the plugin.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)