[GitHub] [hadoop] ayushtkn commented on pull request #3555: HADOOP-17967. Keep restrict-imports-enforcer-rule for Guava VisibleForTesting in hadoop-main pom

[GitBox <gi...@apache.org>]

ayushtkn commented on pull request #3555:
URL: https://github.com/apache/hadoop/pull/3555#issuecomment-946779735


   Just casually exploring, got curious on how does this work. Checked in the PR branch, got one import
   
   https://github.com/virajjasani/hadoop/blob/HADOOP-17967-trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java#L655
   ```
     @org.apache.hadoop.thirdparty.com.google.common.annotations.VisibleForTesting
     int getPriorityLevel(Schedulable e) {
   ```
   Why it is not blocking this & we would need an Addendum somewhere if I am not looking at some old code or have messed up.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org