[
https://issues.apache.org/jira/browse/RANGER-880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15186070#comment-15186070
]
Madhan Neethiraj commented on RANGER-880:
-----------------------------------------
Review board URL: https://reviews.apache.org/r/44538/
> Good coding practice: inner class to be made static; wait() should be in a
> loop
> -------------------------------------------------------------------------------
>
> Key: RANGER-880
> URL: https://issues.apache.org/jira/browse/RANGER-880
> Project: Ranger
> Issue Type: Bug
> Components: tagsync
> Affects Versions: 0.6.0
> Reporter: Madhan Neethiraj
> Assignee: Madhan Neethiraj
> Fix For: 0.6.0
>
> Attachments:
> 0001-RANGER-880-good-coding-practive-inner-class-to-be-ma.patch
>
>
> 1) Inner class TagAdminRESTSink.UploadWorkItem should be made static - as it
> doesn't need access to enclosing class members.
> 2) .wait() method should be called within a loop - to handle possible
> interrupts and spurious wakes - see Java doc for Object.wait() for details.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
