wynot12 commented on a change in pull request #224: [NEMO-375] Clean up of 
remnants of #211
URL: https://github.com/apache/incubator-nemo/pull/224#discussion_r327434806
 
 

 ##########
 File path: 
runtime/master/src/main/java/org/apache/nemo/runtime/master/metric/MetricStore.java
 ##########
 @@ -278,6 +278,10 @@ public void saveOptimizationMetricsToDB(final String 
address, final String jobId
                                           final String dbId, final String 
dbPasswd) {
     final String[] syntax = {"SERIAL PRIMARY KEY"};
 
+    if (MetricUtils.metaDataLoaded()) {
+      MetricUtils.loadMetaData();
+    }
 
 Review comment:
   Why does it call `loadMetaData()` again when metadata is already loaded?
   
   Is the previously loaded metadata stable?
   Then please write a comment.

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


With regards,
Apache Git Services

Reply via email to