You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "Paul Rogers (JIRA)" <ji...@apache.org> on 2018/08/09 07:49:00 UTC

[jira] [Updated] (DRILL-6677) Check style unused import check conflicts with Eclipse

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

Paul Rogers updated DRILL-6677:
-------------------------------
    Description: 
Consider the following Java snippet:

{code}
import com.foo.Bar;

/**
  This is an reference to {@link com.foo.Bar}
{code}

Eclipse will notice the reference to {{com.foo.Bar}} in the Javadoc comment and its automatic import fixer-upper will include the import.

But, check style appears to ignore Javadoc imports. So, Check style reports the import as unused.

The only way, at present, to make Check style happy is to turn off the Eclipse import fixer-upper and do everything manually.

Request: modify check style to also check for class references in Javadoc comments as such references are required for the Javadoc to build.

  was:
Consider the following Java snippet:

{code}
import com.foo.Bar;

/**
  This is an reference to \{\@link com.foo.Bar\}
{code}

Eclipse will notice the reference to {{com.foo.Bar}} in the Javadoc comment and its automatic import fixer-upper will include the import.

But, check style appears to ignore Javadoc imports. So, Check style reports the import as unused.

The only way, at present, to make Check style happy is to turn off the Eclipse import fixer-upper and do everything manually.

Request: modify check style to also check for class references in Javadoc comments as such references are required for the Javadoc to build.


> Check style unused import check conflicts with Eclipse
> ------------------------------------------------------
>
>                 Key: DRILL-6677
>                 URL: https://issues.apache.org/jira/browse/DRILL-6677
>             Project: Apache Drill
>          Issue Type: Bug
>    Affects Versions: 1.15.0
>            Reporter: Paul Rogers
>            Priority: Major
>
> Consider the following Java snippet:
> {code}
> import com.foo.Bar;
> /**
>   This is an reference to {@link com.foo.Bar}
> {code}
> Eclipse will notice the reference to {{com.foo.Bar}} in the Javadoc comment and its automatic import fixer-upper will include the import.
> But, check style appears to ignore Javadoc imports. So, Check style reports the import as unused.
> The only way, at present, to make Check style happy is to turn off the Eclipse import fixer-upper and do everything manually.
> Request: modify check style to also check for class references in Javadoc comments as such references are required for the Javadoc to build.



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