Sergey Edunov created GIRAPH-1069:
-------------------------------------

             Summary: Race condition in all *ConfOption classes
                 Key: GIRAPH-1069
                 URL: https://issues.apache.org/jira/browse/GIRAPH-1069
             Project: Giraph
          Issue Type: Bug
            Reporter: Sergey Edunov
            Assignee: Sergey Edunov


*ConfOption classes, such as ClassConfOption, IntConfOption, FloatConfOption 
etc, call AllOtions.add(this) from their constructor. This call updates static 
list without any synchronization. Hence, if you create conf option classes in 
different threads you run into race condition. 

The only reason we have AllOptions is to create documentation. It can be done 
with reflection instead. So, let's remove AllOtions.add(this) from all conf 
classes and implement reflection based approach in AllOptions



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to