[
https://issues.apache.org/jira/browse/CURATOR-530?focusedWorklogId=275487&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-275487
]
ASF GitHub Bot logged work on CURATOR-530:
------------------------------------------
Author: ASF GitHub Bot
Created on: 11/Jul/19 19:33
Start Date: 11/Jul/19 19:33
Worklog Time Spent: 10m
Work Description: shayshim commented on pull request #315: [CURATOR-530]
Documentation on InterProcessSemaphoreMutex is misleading
URL: https://github.com/apache/curator/pull/315#discussion_r302708567
##########
File path:
curator-recipes/src/main/java/org/apache/curator/framework/recipes/locks/InterProcessMultiLock.java
##########
@@ -145,6 +145,9 @@ public boolean acquire(long time, TimeUnit unit) throws
Exception
* {@inheritDoc}
*
* <p>NOTE: locks are released in the reverse order that they were
acquired.</p>
+ *
+ * @throws Exception ZK errors, interruptions, current thread does not own
the lock
Review comment:
@emretetik96 right, my mistake
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 275487)
Time Spent: 50m (was: 40m)
> Documentation on InterProcessSemaphoreMutex is misleading
> ---------------------------------------------------------
>
> Key: CURATOR-530
> URL: https://issues.apache.org/jira/browse/CURATOR-530
> Project: Apache Curator
> Issue Type: Bug
> Components: Documentation
> Affects Versions: 4.0.1
> Reporter: Emre Tetik
> Assignee: Shay Shimony
> Priority: Major
> Labels: documentation, easyfix, pull-request-available
> Time Spent: 50m
> Remaining Estimate: 0h
>
> The documentation for the release() method in InterProcessSemaphoreMutex says
> that it will throw an exception if the method is called on a thread other
> than the one which acquired the lock:
> [https://curator.apache.org/apidocs/org/apache/curator/framework/recipes/locks/InterProcessSemaphoreMutex.html#release--]
> This is not correct, and should be changed.
--
This message was sent by Atlassian JIRA
(v7.6.14#76016)
