[afs] Don't lock shared data whilst fetching remote data

Since the fetch can be slow, there's no need to lock other afs threads while we wait for a particular set of results to be fetched.
qgis · Feb 20, 2018 · 7062668 · 7062668
1 parent 7307650
commit 7062668
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/providers/arcgisrest/qgsafsshareddata.cpp b/src/providers/arcgisrest/qgsafsshareddata.cpp
@@ -62,6 +62,9 @@ bool QgsAfsSharedData::getFeature( QgsFeatureId id, QgsFeature &f, const QgsRect
     return false;
   }
 
+  // don't lock while doing the fetch
+  locker.unlock();
+
   // Query
   QString errorTitle, errorMessage;
   const QVariantMap queryData = QgsArcGisRestUtils::getObjects(
@@ -75,6 +78,8 @@ bool QgsAfsSharedData::getFeature( QgsFeatureId id, QgsFeature &f, const QgsRect
     return false;
   }
 
+  // but re-lock while updating cache
+  locker.relock();
   const QVariantList featuresData = queryData[QStringLiteral( "features" )].toList();
   if ( featuresData.isEmpty() )
   {