[
https://issues.apache.org/jira/browse/HADOOP-18196?focusedWorklogId=757409&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-757409
]
ASF GitHub Bot logged work on HADOOP-18196:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 15/Apr/22 14:17
Start Date: 15/Apr/22 14:17
Worklog Time Spent: 10m
Work Description: virajjasani commented on PR #4152:
URL: https://github.com/apache/hadoop/pull/4152#issuecomment-1100136074
@aajisaka @tasanuma could you please take a look?
Issue Time Tracking
-------------------
Worklog Id: (was: 757409)
Time Spent: 40m (was: 0.5h)
> Remove replace-guava from replacer plugin
> -----------------------------------------
>
> Key: HADOOP-18196
> URL: https://issues.apache.org/jira/browse/HADOOP-18196
> Project: Hadoop Common
> Issue Type: Task
> Reporter: Viraj Jasani
> Assignee: Viraj Jasani
> Priority: Major
> Labels: pull-request-available
> Time Spent: 40m
> Remaining Estimate: 0h
>
> While running the build, realized that all replacer plugin executions run
> only after "banned-illegal-imports" enforcer plugin.
> For instance,
> {code:java}
> [INFO] --- maven-enforcer-plugin:3.0.0:enforce (banned-illegal-imports) @
> hadoop-cloud-storage ---
> [INFO]
> [INFO] --- replacer:1.5.3:replace (replace-generated-sources) @
> hadoop-cloud-storage ---
> [INFO] Skipping
> [INFO]
> [INFO] --- replacer:1.5.3:replace (replace-sources) @ hadoop-cloud-storage ---
> [INFO] Skipping
> [INFO]
> [INFO] --- replacer:1.5.3:replace (replace-guava) @ hadoop-cloud-storage ---
> [INFO] Replacement run on 0 file.
> [INFO] {code}
> Hence, if our source code uses com.google.common, banned-illegal-imports will
> cause the build failure and replacer plugin would not even get executed.
> We should remove it as it is only redundant execution step.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
