paulazomig commented on code in PR #6408:
URL: https://github.com/apache/cloudstack/pull/6408#discussion_r889246860


##########
engine/storage/volume/src/main/java/org/apache/cloudstack/storage/datastore/PrimaryDataStoreImpl.java:
##########
@@ -91,6 +91,9 @@ public class PrimaryDataStoreImpl implements PrimaryDataStore 
{
     private VolumeDao volumeDao;
     private Map<String, String> _details;
 
+    private String uuid;

Review Comment:
   @DaanHoogland, sorry, my explanation was not clear in the first comment.
   
   Those fields were added this way because we're using 
`ReflectionToStringBuilderUtils` to build the string to print in the log. As 
`ReflectionToStringBuilderUtils` uses the object attributes to build the 
string, this object is declared as an interface `StoragePool` (in runtime this 
object is an instance of `PrimaryDataStoreImpl`) and we cannot retrieve the 
`StoragePoolVO` contained in it, we declared these attributes (that already are 
there) to be reflected later.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to