Re: JAVADOC fails on local build. Should it be checked on TC?
I updated javadoc TC job [1]. Completes successfully on master so far. [1] https://ci.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Javadoc?branch=%3Cdefault%3E=overview=builds Best regards, Ivan Pavlukhin пт, 14 февр. 2020 г. в 16:23, Ivan Pavlukhin : > > I merged changes for [1] to master. Next steps will be updating TC > javadoc to fail on a string "Javadoc Warnings" > > [1] https://issues.apache.org/jira/browse/IGNITE-12644 > > Best regards, > Ivan Pavlukhin > > сб, 8 февр. 2020 г. в 21:03, Ivan Pavlukhin : > > > > I created a ticket [1] and PR [2]. Please review. > > > > [1] https://issues.apache.org/jira/browse/IGNITE-12644 > > [2] https://github.com/apache/ignite/pull/7356 > > > > Best regards, > > Ivan Pavlukhin > > > > вт, 4 февр. 2020 г. в 17:51, Ivan Pavlukhin : > > > > > > The next bit I found out is that links like > > > "https://hadoop.apache.org/docs/current/api/" > > > are needed to have working links to external classes in javadocs (e.g. > > > Hadoop ones). Unfortunately, we are not consistent with external > > > links. E.g. references to JCache classes are rendered as plain text in > > > our javadocs [1]. > > > > > > For now I suggest to drop Hadoop links from maven plugin configuration > > > as well. If there are no objections I will prepare a PR. > > > > > > [1] > > > https://ignite.apache.org/releases/latest/javadoc/org/apache/ignite/cache/CacheEntryProcessor.html > > > > > > вт, 4 февр. 2020 г. в 15:04, Ivan Pavlukhin : > > > > > > > > Alexey, > > > > > > > > Thank you for the hint. Actually my bet was the same. But currently I > > > > have no understanding how > > > > "https://hadoop.apache.org/docs/current/api/" tag is used > > > > and is it used at all. > > > > > > > > вт, 4 февр. 2020 г. в 15:01, Alexey Goncharuk > > > > : > > > > > > > > > > Ivan, > > > > > > > > > > My guess would be that the link was included for the Hadoop > > > > > Accelerator > > > > > which we all agreed to remove [1]. I think it is safe to remove it > > > > > from > > > > > pom.xml completely. > > > > > > > > > > [1] > > > > > http://apache-ignite-developers.2346864.n4.nabble.com/DISCUSSION-Complete-Discontinuation-of-IGFS-and-Hadoop-Accelerator-td42282.html > > > > > > > > > > > > > > > > -- > > > > Best regards, > > > > Ivan Pavlukhin > > > > > > > > > > > > -- > > > Best regards, > > > Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
I merged changes for [1] to master. Next steps will be updating TC javadoc to fail on a string "Javadoc Warnings" [1] https://issues.apache.org/jira/browse/IGNITE-12644 Best regards, Ivan Pavlukhin сб, 8 февр. 2020 г. в 21:03, Ivan Pavlukhin : > > I created a ticket [1] and PR [2]. Please review. > > [1] https://issues.apache.org/jira/browse/IGNITE-12644 > [2] https://github.com/apache/ignite/pull/7356 > > Best regards, > Ivan Pavlukhin > > вт, 4 февр. 2020 г. в 17:51, Ivan Pavlukhin : > > > > The next bit I found out is that links like > > "https://hadoop.apache.org/docs/current/api/" > > are needed to have working links to external classes in javadocs (e.g. > > Hadoop ones). Unfortunately, we are not consistent with external > > links. E.g. references to JCache classes are rendered as plain text in > > our javadocs [1]. > > > > For now I suggest to drop Hadoop links from maven plugin configuration > > as well. If there are no objections I will prepare a PR. > > > > [1] > > https://ignite.apache.org/releases/latest/javadoc/org/apache/ignite/cache/CacheEntryProcessor.html > > > > вт, 4 февр. 2020 г. в 15:04, Ivan Pavlukhin : > > > > > > Alexey, > > > > > > Thank you for the hint. Actually my bet was the same. But currently I > > > have no understanding how > > > "https://hadoop.apache.org/docs/current/api/" tag is used > > > and is it used at all. > > > > > > вт, 4 февр. 2020 г. в 15:01, Alexey Goncharuk > > > : > > > > > > > > Ivan, > > > > > > > > My guess would be that the link was included for the Hadoop Accelerator > > > > which we all agreed to remove [1]. I think it is safe to remove it from > > > > pom.xml completely. > > > > > > > > [1] > > > > http://apache-ignite-developers.2346864.n4.nabble.com/DISCUSSION-Complete-Discontinuation-of-IGFS-and-Hadoop-Accelerator-td42282.html > > > > > > > > > > > > -- > > > Best regards, > > > Ivan Pavlukhin > > > > > > > > -- > > Best regards, > > Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
I created a ticket [1] and PR [2]. Please review. [1] https://issues.apache.org/jira/browse/IGNITE-12644 [2] https://github.com/apache/ignite/pull/7356 Best regards, Ivan Pavlukhin вт, 4 февр. 2020 г. в 17:51, Ivan Pavlukhin : > > The next bit I found out is that links like > "https://hadoop.apache.org/docs/current/api/" > are needed to have working links to external classes in javadocs (e.g. > Hadoop ones). Unfortunately, we are not consistent with external > links. E.g. references to JCache classes are rendered as plain text in > our javadocs [1]. > > For now I suggest to drop Hadoop links from maven plugin configuration > as well. If there are no objections I will prepare a PR. > > [1] > https://ignite.apache.org/releases/latest/javadoc/org/apache/ignite/cache/CacheEntryProcessor.html > > вт, 4 февр. 2020 г. в 15:04, Ivan Pavlukhin : > > > > Alexey, > > > > Thank you for the hint. Actually my bet was the same. But currently I > > have no understanding how > > "https://hadoop.apache.org/docs/current/api/" tag is used > > and is it used at all. > > > > вт, 4 февр. 2020 г. в 15:01, Alexey Goncharuk : > > > > > > Ivan, > > > > > > My guess would be that the link was included for the Hadoop Accelerator > > > which we all agreed to remove [1]. I think it is safe to remove it from > > > pom.xml completely. > > > > > > [1] > > > http://apache-ignite-developers.2346864.n4.nabble.com/DISCUSSION-Complete-Discontinuation-of-IGFS-and-Hadoop-Accelerator-td42282.html > > > > > > > > -- > > Best regards, > > Ivan Pavlukhin > > > > -- > Best regards, > Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
The next bit I found out is that links like "https://hadoop.apache.org/docs/current/api/" are needed to have working links to external classes in javadocs (e.g. Hadoop ones). Unfortunately, we are not consistent with external links. E.g. references to JCache classes are rendered as plain text in our javadocs [1]. For now I suggest to drop Hadoop links from maven plugin configuration as well. If there are no objections I will prepare a PR. [1] https://ignite.apache.org/releases/latest/javadoc/org/apache/ignite/cache/CacheEntryProcessor.html вт, 4 февр. 2020 г. в 15:04, Ivan Pavlukhin : > > Alexey, > > Thank you for the hint. Actually my bet was the same. But currently I > have no understanding how > "https://hadoop.apache.org/docs/current/api/" tag is used > and is it used at all. > > вт, 4 февр. 2020 г. в 15:01, Alexey Goncharuk : > > > > Ivan, > > > > My guess would be that the link was included for the Hadoop Accelerator > > which we all agreed to remove [1]. I think it is safe to remove it from > > pom.xml completely. > > > > [1] > > http://apache-ignite-developers.2346864.n4.nabble.com/DISCUSSION-Complete-Discontinuation-of-IGFS-and-Hadoop-Accelerator-td42282.html > > > > -- > Best regards, > Ivan Pavlukhin -- Best regards, Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
Alexey, Thank you for the hint. Actually my bet was the same. But currently I have no understanding how "https://hadoop.apache.org/docs/current/api/" tag is used and is it used at all. вт, 4 февр. 2020 г. в 15:01, Alexey Goncharuk : > > Ivan, > > My guess would be that the link was included for the Hadoop Accelerator > which we all agreed to remove [1]. I think it is safe to remove it from > pom.xml completely. > > [1] > http://apache-ignite-developers.2346864.n4.nabble.com/DISCUSSION-Complete-Discontinuation-of-IGFS-and-Hadoop-Accelerator-td42282.html -- Best regards, Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
Ivan, My guess would be that the link was included for the Hadoop Accelerator which we all agreed to remove [1]. I think it is safe to remove it from pom.xml completely. [1] http://apache-ignite-developers.2346864.n4.nabble.com/DISCUSSION-Complete-Discontinuation-of-IGFS-and-Hadoop-Accelerator-td42282.html
Re: JAVADOC fails on local build. Should it be checked on TC?
I continued my experiments with javadocs and a next hurdle was an external link to Hadoop documentation https://hadoop.apache.org/docs/current/api/ . Javadoc build job scans logs for javadoc-specific warnings and it founds any it fails build. But there were many failures in the past in a form: "javadoc: warning - Error fetching URL: https://hadoop.apache.org/docs/current/api/; Consequently we ignore such lines. But a check we do seems unfair as we currently have malformed javadocs. Currently we use a following regexp to find javadoc warnings: "^\[WARNING\] javadoc: warning - (?!Error fetching URL).*$" I suppose a proper strategy is to search for "Javadoc Warnings" substring. But we must deal with a Hadoop external link. I made an experiment and removed "https://hadoop.apache.org/docs/current/api/" tag from parent/pom.xml and fixed all other existed javadoc warnings. I received a green build result [1] (in an experimental build job). Do we really need this link to Hadoop javadocs? What do you think? [1] https://ci.ignite.apache.org/buildConfiguration/IgniteTests24Java8_ExperimentJavadoc?branch=pull%2F7356%2Fhead=overview=builds пт, 24 янв. 2020 г. в 16:06, Ivan Pavlukhin : > > Actually, I got stuck with a question what do modules like > ignite-hibernate mean from a javadoc standpoint. It seems that modules > sharing same packages and classes cannot be aggregated into valid > javadoc bundle. And actually this problem should be solved as well > (modularization?). But I suppose that it is closer to a release > process rather than daily TC check. > > So, my current understanding is: > 1. Javadoc job should be fixed to reveal problems (with code links). > 2. "javadoc" profile is not needed for daily TC check, but it > highlights some release-related problems. > > пт, 24 янв. 2020 г. в 15:44, Maxim Muzafarov : > > > > Folks, > > > > > > It's true that my simple assumption (to include Javadoc profile) will > > not work by reasons mentioned by Petr. I've tested it under a new > > experimental suite [1] which I've created earlier. We will always have > > such warnings due to we have multiple modules with the same package > > declarations (e.g. ignite-hibernate_5.1, hibernate_4.2, hibernate_5.3 > > modules and the same for others like spring-data): > > > > [WARNING] Javadoc: warning - Multiple sources of package comments > > found for package "org.apache.ignite.cache.store.hibernate" > > > > So, if we build all modules it's not correct to fail the whole JavaDoc > > suite by existing such warnings in the suite logs. > > > > > > I'm still thinking of how can we handle all such cases and I don't > > have an easy solution for now. At least we can do checks with two > > steps: > > 1. build with Javadoc profile ignore warnings (fail on an error) > > 2. build and check that there is no warnings for ignite-core module > > > > Thoughts? > > > > > > [1] > > https://ci.ignite.apache.org/viewLog.html?buildId=4927599=IgniteTests24Java8_JavadocExtended > > > > > > > > On Fri, 24 Jan 2020 at 13:47, Ivan Pavlukhin wrote: > > > > > > Petr, > > > > > > Thank you for that detail! Unfortunately I do not know how to fix it > > > easily. > > > > > > Also I found that even without "javadoc" profile we miss some javadoc > > > problems. I believe that it can be fixed by updating post-build > > > javadoc checking condition. See below what problems do we miss (and > > > there are a bit more in full logs): > > > [05:08:51]W: [Step 2/2] [WARNING] Javadoc Warnings > > > [05:08:51]W: [Step 2/2] [WARNING] > > > /opt/buildagent/work/7bc1c54bc719b67c/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java:2351: > > > warning - Tag @link: reference not found: > > > GridDistributedCacheAdapter.GlobalRemoveAllJob > > > [05:08:51]W: [Step 2/2] [WARNING] > > > /opt/buildagent/work/7bc1c54bc719b67c/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java:2351: > > > warning - Tag @link: reference not found: > > > GridDistributedCacheAdapter.GlobalRemoveAllJob > > > > > > пт, 24 янв. 2020 г. в 11:56, Petr Ivanov : > > > > > > > > We could not because of multiple hibernate modules, what is considered > > > > from the javadoc's point as warning, that always fails test. > > > > > > > > > > > > > On 24 Jan 2020, at 11:02, Ivan Pavlukhin wrote: > > > > > > > > > > Maxim, > > > > > > > > > > I suppose we can try to enable "javadoc" profile for a corresponding > > > > > job. Are there any objections to do so? > > > > > > > > > > пн, 20 янв. 2020 г. в 13:25, Ivan Pavlukhin : > > > > >> > > > > >> Maxim, > > > > >> > > > > >> From the first glance it seems that "javadoc" profile was really > > > > >> missed. Are there any other problems except springdata22? If no then > > > > >> we can add the profile. Also it is interesting how it influence on > > > > >> execution time? > > > > >> > > > > >> пн, 13 янв. 2020 г. в 16:53, Maxim Muzafarov : > > > > >>> > > > > >>> Igniters,
Re: JAVADOC fails on local build. Should it be checked on TC?
Actually, I got stuck with a question what do modules like ignite-hibernate mean from a javadoc standpoint. It seems that modules sharing same packages and classes cannot be aggregated into valid javadoc bundle. And actually this problem should be solved as well (modularization?). But I suppose that it is closer to a release process rather than daily TC check. So, my current understanding is: 1. Javadoc job should be fixed to reveal problems (with code links). 2. "javadoc" profile is not needed for daily TC check, but it highlights some release-related problems. пт, 24 янв. 2020 г. в 15:44, Maxim Muzafarov : > > Folks, > > > It's true that my simple assumption (to include Javadoc profile) will > not work by reasons mentioned by Petr. I've tested it under a new > experimental suite [1] which I've created earlier. We will always have > such warnings due to we have multiple modules with the same package > declarations (e.g. ignite-hibernate_5.1, hibernate_4.2, hibernate_5.3 > modules and the same for others like spring-data): > > [WARNING] Javadoc: warning - Multiple sources of package comments > found for package "org.apache.ignite.cache.store.hibernate" > > So, if we build all modules it's not correct to fail the whole JavaDoc > suite by existing such warnings in the suite logs. > > > I'm still thinking of how can we handle all such cases and I don't > have an easy solution for now. At least we can do checks with two > steps: > 1. build with Javadoc profile ignore warnings (fail on an error) > 2. build and check that there is no warnings for ignite-core module > > Thoughts? > > > [1] > https://ci.ignite.apache.org/viewLog.html?buildId=4927599=IgniteTests24Java8_JavadocExtended > > > > On Fri, 24 Jan 2020 at 13:47, Ivan Pavlukhin wrote: > > > > Petr, > > > > Thank you for that detail! Unfortunately I do not know how to fix it easily. > > > > Also I found that even without "javadoc" profile we miss some javadoc > > problems. I believe that it can be fixed by updating post-build > > javadoc checking condition. See below what problems do we miss (and > > there are a bit more in full logs): > > [05:08:51]W: [Step 2/2] [WARNING] Javadoc Warnings > > [05:08:51]W: [Step 2/2] [WARNING] > > /opt/buildagent/work/7bc1c54bc719b67c/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java:2351: > > warning - Tag @link: reference not found: > > GridDistributedCacheAdapter.GlobalRemoveAllJob > > [05:08:51]W: [Step 2/2] [WARNING] > > /opt/buildagent/work/7bc1c54bc719b67c/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java:2351: > > warning - Tag @link: reference not found: > > GridDistributedCacheAdapter.GlobalRemoveAllJob > > > > пт, 24 янв. 2020 г. в 11:56, Petr Ivanov : > > > > > > We could not because of multiple hibernate modules, what is considered > > > from the javadoc's point as warning, that always fails test. > > > > > > > > > > On 24 Jan 2020, at 11:02, Ivan Pavlukhin wrote: > > > > > > > > Maxim, > > > > > > > > I suppose we can try to enable "javadoc" profile for a corresponding > > > > job. Are there any objections to do so? > > > > > > > > пн, 20 янв. 2020 г. в 13:25, Ivan Pavlukhin : > > > >> > > > >> Maxim, > > > >> > > > >> From the first glance it seems that "javadoc" profile was really > > > >> missed. Are there any other problems except springdata22? If no then > > > >> we can add the profile. Also it is interesting how it influence on > > > >> execution time? > > > >> > > > >> пн, 13 янв. 2020 г. в 16:53, Maxim Muzafarov : > > > >>> > > > >>> Igniters, > > > >>> > > > >>> > > > >>> I've run locally maven command according to DEVNOTES: > > > >>> > > > >>> mvn initialize -Pjavadoc > > > >>> > > > >>> and it fails due to: 'Other Packages' section should not be present, > > > >>> all packages should have corresponding documentation groups. The > > > >>> reason of that is a newly added `org.apache.ignite.springdata22` > > > >>> package [1] is missing in maven-javadoc-plugin configuration [2]. > > > >>> > > > >>> > > > >>> We have Javadoc Suite [3] but it not checks such issues due to > > > >>> `javadoc` maven profile required to be enabled. > > > >>> Should we enable `javadoc` profile for this suite? > > > >>> Any other thoughts? > > > >>> > > > >>> [1] https://issues.apache.org/jira/browse/IGNITE-12259 > > > >>> [2] https://issues.apache.org/jira/browse/IGNITE-12528 > > > >>> [3] > > > >>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_Javadoc_IgniteTests24Java8=%3Cdefault%3E=buildTypeStatusDiv > > > >> > > > >> > > > >> > > > >> -- > > > >> Best regards, > > > >> Ivan Pavlukhin > > > > > > > > > > > > > > > > -- > > > > Best regards, > > > > Ivan Pavlukhin > > > > > > > > > -- > > Best regards, > > Ivan Pavlukhin -- Best regards, Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
Folks, It's true that my simple assumption (to include Javadoc profile) will not work by reasons mentioned by Petr. I've tested it under a new experimental suite [1] which I've created earlier. We will always have such warnings due to we have multiple modules with the same package declarations (e.g. ignite-hibernate_5.1, hibernate_4.2, hibernate_5.3 modules and the same for others like spring-data): [WARNING] Javadoc: warning - Multiple sources of package comments found for package "org.apache.ignite.cache.store.hibernate" So, if we build all modules it's not correct to fail the whole JavaDoc suite by existing such warnings in the suite logs. I'm still thinking of how can we handle all such cases and I don't have an easy solution for now. At least we can do checks with two steps: 1. build with Javadoc profile ignore warnings (fail on an error) 2. build and check that there is no warnings for ignite-core module Thoughts? [1] https://ci.ignite.apache.org/viewLog.html?buildId=4927599=IgniteTests24Java8_JavadocExtended On Fri, 24 Jan 2020 at 13:47, Ivan Pavlukhin wrote: > > Petr, > > Thank you for that detail! Unfortunately I do not know how to fix it easily. > > Also I found that even without "javadoc" profile we miss some javadoc > problems. I believe that it can be fixed by updating post-build > javadoc checking condition. See below what problems do we miss (and > there are a bit more in full logs): > [05:08:51]W: [Step 2/2] [WARNING] Javadoc Warnings > [05:08:51]W: [Step 2/2] [WARNING] > /opt/buildagent/work/7bc1c54bc719b67c/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java:2351: > warning - Tag @link: reference not found: > GridDistributedCacheAdapter.GlobalRemoveAllJob > [05:08:51]W: [Step 2/2] [WARNING] > /opt/buildagent/work/7bc1c54bc719b67c/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java:2351: > warning - Tag @link: reference not found: > GridDistributedCacheAdapter.GlobalRemoveAllJob > > пт, 24 янв. 2020 г. в 11:56, Petr Ivanov : > > > > We could not because of multiple hibernate modules, what is considered from > > the javadoc's point as warning, that always fails test. > > > > > > > On 24 Jan 2020, at 11:02, Ivan Pavlukhin wrote: > > > > > > Maxim, > > > > > > I suppose we can try to enable "javadoc" profile for a corresponding > > > job. Are there any objections to do so? > > > > > > пн, 20 янв. 2020 г. в 13:25, Ivan Pavlukhin : > > >> > > >> Maxim, > > >> > > >> From the first glance it seems that "javadoc" profile was really > > >> missed. Are there any other problems except springdata22? If no then > > >> we can add the profile. Also it is interesting how it influence on > > >> execution time? > > >> > > >> пн, 13 янв. 2020 г. в 16:53, Maxim Muzafarov : > > >>> > > >>> Igniters, > > >>> > > >>> > > >>> I've run locally maven command according to DEVNOTES: > > >>> > > >>> mvn initialize -Pjavadoc > > >>> > > >>> and it fails due to: 'Other Packages' section should not be present, > > >>> all packages should have corresponding documentation groups. The > > >>> reason of that is a newly added `org.apache.ignite.springdata22` > > >>> package [1] is missing in maven-javadoc-plugin configuration [2]. > > >>> > > >>> > > >>> We have Javadoc Suite [3] but it not checks such issues due to > > >>> `javadoc` maven profile required to be enabled. > > >>> Should we enable `javadoc` profile for this suite? > > >>> Any other thoughts? > > >>> > > >>> [1] https://issues.apache.org/jira/browse/IGNITE-12259 > > >>> [2] https://issues.apache.org/jira/browse/IGNITE-12528 > > >>> [3] > > >>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_Javadoc_IgniteTests24Java8=%3Cdefault%3E=buildTypeStatusDiv > > >> > > >> > > >> > > >> -- > > >> Best regards, > > >> Ivan Pavlukhin > > > > > > > > > > > > -- > > > Best regards, > > > Ivan Pavlukhin > > > > > -- > Best regards, > Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
Petr, Thank you for that detail! Unfortunately I do not know how to fix it easily. Also I found that even without "javadoc" profile we miss some javadoc problems. I believe that it can be fixed by updating post-build javadoc checking condition. See below what problems do we miss (and there are a bit more in full logs): [05:08:51]W: [Step 2/2] [WARNING] Javadoc Warnings [05:08:51]W: [Step 2/2] [WARNING] /opt/buildagent/work/7bc1c54bc719b67c/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java:2351: warning - Tag @link: reference not found: GridDistributedCacheAdapter.GlobalRemoveAllJob [05:08:51]W: [Step 2/2] [WARNING] /opt/buildagent/work/7bc1c54bc719b67c/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java:2351: warning - Tag @link: reference not found: GridDistributedCacheAdapter.GlobalRemoveAllJob пт, 24 янв. 2020 г. в 11:56, Petr Ivanov : > > We could not because of multiple hibernate modules, what is considered from > the javadoc's point as warning, that always fails test. > > > > On 24 Jan 2020, at 11:02, Ivan Pavlukhin wrote: > > > > Maxim, > > > > I suppose we can try to enable "javadoc" profile for a corresponding > > job. Are there any objections to do so? > > > > пн, 20 янв. 2020 г. в 13:25, Ivan Pavlukhin : > >> > >> Maxim, > >> > >> From the first glance it seems that "javadoc" profile was really > >> missed. Are there any other problems except springdata22? If no then > >> we can add the profile. Also it is interesting how it influence on > >> execution time? > >> > >> пн, 13 янв. 2020 г. в 16:53, Maxim Muzafarov : > >>> > >>> Igniters, > >>> > >>> > >>> I've run locally maven command according to DEVNOTES: > >>> > >>> mvn initialize -Pjavadoc > >>> > >>> and it fails due to: 'Other Packages' section should not be present, > >>> all packages should have corresponding documentation groups. The > >>> reason of that is a newly added `org.apache.ignite.springdata22` > >>> package [1] is missing in maven-javadoc-plugin configuration [2]. > >>> > >>> > >>> We have Javadoc Suite [3] but it not checks such issues due to > >>> `javadoc` maven profile required to be enabled. > >>> Should we enable `javadoc` profile for this suite? > >>> Any other thoughts? > >>> > >>> [1] https://issues.apache.org/jira/browse/IGNITE-12259 > >>> [2] https://issues.apache.org/jira/browse/IGNITE-12528 > >>> [3] > >>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_Javadoc_IgniteTests24Java8=%3Cdefault%3E=buildTypeStatusDiv > >> > >> > >> > >> -- > >> Best regards, > >> Ivan Pavlukhin > > > > > > > > -- > > Best regards, > > Ivan Pavlukhin > -- Best regards, Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
We could not because of multiple hibernate modules, what is considered from the javadoc's point as warning, that always fails test. > On 24 Jan 2020, at 11:02, Ivan Pavlukhin wrote: > > Maxim, > > I suppose we can try to enable "javadoc" profile for a corresponding > job. Are there any objections to do so? > > пн, 20 янв. 2020 г. в 13:25, Ivan Pavlukhin : >> >> Maxim, >> >> From the first glance it seems that "javadoc" profile was really >> missed. Are there any other problems except springdata22? If no then >> we can add the profile. Also it is interesting how it influence on >> execution time? >> >> пн, 13 янв. 2020 г. в 16:53, Maxim Muzafarov : >>> >>> Igniters, >>> >>> >>> I've run locally maven command according to DEVNOTES: >>> >>> mvn initialize -Pjavadoc >>> >>> and it fails due to: 'Other Packages' section should not be present, >>> all packages should have corresponding documentation groups. The >>> reason of that is a newly added `org.apache.ignite.springdata22` >>> package [1] is missing in maven-javadoc-plugin configuration [2]. >>> >>> >>> We have Javadoc Suite [3] but it not checks such issues due to >>> `javadoc` maven profile required to be enabled. >>> Should we enable `javadoc` profile for this suite? >>> Any other thoughts? >>> >>> [1] https://issues.apache.org/jira/browse/IGNITE-12259 >>> [2] https://issues.apache.org/jira/browse/IGNITE-12528 >>> [3] >>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_Javadoc_IgniteTests24Java8=%3Cdefault%3E=buildTypeStatusDiv >> >> >> >> -- >> Best regards, >> Ivan Pavlukhin > > > > -- > Best regards, > Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
Maxim, I suppose we can try to enable "javadoc" profile for a corresponding job. Are there any objections to do so? пн, 20 янв. 2020 г. в 13:25, Ivan Pavlukhin : > > Maxim, > > From the first glance it seems that "javadoc" profile was really > missed. Are there any other problems except springdata22? If no then > we can add the profile. Also it is interesting how it influence on > execution time? > > пн, 13 янв. 2020 г. в 16:53, Maxim Muzafarov : > > > > Igniters, > > > > > > I've run locally maven command according to DEVNOTES: > > > > mvn initialize -Pjavadoc > > > > and it fails due to: 'Other Packages' section should not be present, > > all packages should have corresponding documentation groups. The > > reason of that is a newly added `org.apache.ignite.springdata22` > > package [1] is missing in maven-javadoc-plugin configuration [2]. > > > > > > We have Javadoc Suite [3] but it not checks such issues due to > > `javadoc` maven profile required to be enabled. > > Should we enable `javadoc` profile for this suite? > > Any other thoughts? > > > > [1] https://issues.apache.org/jira/browse/IGNITE-12259 > > [2] https://issues.apache.org/jira/browse/IGNITE-12528 > > [3] > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_Javadoc_IgniteTests24Java8=%3Cdefault%3E=buildTypeStatusDiv > > > > -- > Best regards, > Ivan Pavlukhin -- Best regards, Ivan Pavlukhin
Re: JAVADOC fails on local build. Should it be checked on TC?
Maxim, >From the first glance it seems that "javadoc" profile was really missed. Are there any other problems except springdata22? If no then we can add the profile. Also it is interesting how it influence on execution time? пн, 13 янв. 2020 г. в 16:53, Maxim Muzafarov : > > Igniters, > > > I've run locally maven command according to DEVNOTES: > > mvn initialize -Pjavadoc > > and it fails due to: 'Other Packages' section should not be present, > all packages should have corresponding documentation groups. The > reason of that is a newly added `org.apache.ignite.springdata22` > package [1] is missing in maven-javadoc-plugin configuration [2]. > > > We have Javadoc Suite [3] but it not checks such issues due to > `javadoc` maven profile required to be enabled. > Should we enable `javadoc` profile for this suite? > Any other thoughts? > > [1] https://issues.apache.org/jira/browse/IGNITE-12259 > [2] https://issues.apache.org/jira/browse/IGNITE-12528 > [3] > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_Javadoc_IgniteTests24Java8=%3Cdefault%3E=buildTypeStatusDiv -- Best regards, Ivan Pavlukhin
JAVADOC fails on local build. Should it be checked on TC?
Igniters, I've run locally maven command according to DEVNOTES: mvn initialize -Pjavadoc and it fails due to: 'Other Packages' section should not be present, all packages should have corresponding documentation groups. The reason of that is a newly added `org.apache.ignite.springdata22` package [1] is missing in maven-javadoc-plugin configuration [2]. We have Javadoc Suite [3] but it not checks such issues due to `javadoc` maven profile required to be enabled. Should we enable `javadoc` profile for this suite? Any other thoughts? [1] https://issues.apache.org/jira/browse/IGNITE-12259 [2] https://issues.apache.org/jira/browse/IGNITE-12528 [3] https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_Javadoc_IgniteTests24Java8=%3Cdefault%3E=buildTypeStatusDiv