[ 
https://issues.apache.org/jira/browse/SENTRY-2237?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Na Li updated SENTRY-2237:
--------------------------
    Description: 
My patch had a build error, but when build script applied the patch, instead of 
showing the error message, it shows 
{code}
error: 
a/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/service/common/ServiceConstants.java:
 does not exist in index
{code}
This error is caused by "patch -p0", then the script tries "patch -p1", which 
succeded. We should suppress this error.

The desired behavior is to output the direct cause of the failure
{code}
 PMD Failure: org.apache.sentry.provider.db.service.persistent.SentryStore:828 
Rule:UnusedFormalParameter Priority:3 Avoid unused method parameters such as 
'pm'.
{code}

  was:
My patch had a build error, but when build script applied the patch, instead of 
showing the error message, it shows 
{code}
error: 
a/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/service/common/ServiceConstants.java:
 does not exist in index
{code}

The desired behavior is to output the direct cause of the failure
{code}
 PMD Failure: org.apache.sentry.provider.db.service.persistent.SentryStore:828 
Rule:UnusedFormalParameter Priority:3 Avoid unused method parameters such as 
'pm'.
{code}


> Error message of p0 is not intuitive
> ------------------------------------
>
>                 Key: SENTRY-2237
>                 URL: https://issues.apache.org/jira/browse/SENTRY-2237
>             Project: Sentry
>          Issue Type: Bug
>          Components: Sentry
>    Affects Versions: 2.1.0
>            Reporter: Na Li
>            Priority: Major
>              Labels: newbie,
>         Attachments: SENTRY-2237 Console output.txt, Sentry-2237 artifacts.zip
>
>
> My patch had a build error, but when build script applied the patch, instead 
> of showing the error message, it shows 
> {code}
> error: 
> a/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/service/common/ServiceConstants.java:
>  does not exist in index
> {code}
> This error is caused by "patch -p0", then the script tries "patch -p1", which 
> succeded. We should suppress this error.
> The desired behavior is to output the direct cause of the failure
> {code}
>  PMD Failure: 
> org.apache.sentry.provider.db.service.persistent.SentryStore:828 
> Rule:UnusedFormalParameter Priority:3 Avoid unused method parameters such as 
> 'pm'.
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to