Fix --job=test

When the define_py_data_sources2 has both train_list and test_list,
for job=test, the trainer will create both dataProvider_ and testDataProvider_.
But dataProvider_ is not used. This causes SIGSEGV at finishAsync() because asyncLoader_ is not created.

Change-Id: If579f715f80a70ebc795094792c3436bfa0f5746

paddle/gserver/dataproviders/DataProvider.h

@@ -271,8 +271,10 @@ public:
   void finishAsyncLoad() {
     stopping_ = true;
     taskReadySem_.post();
+    if (asyncLoader_) {
       asyncLoader_->join();
     }
+  }
 
   void setPending(bool pending) { pending_ = pending; }
 

paddle/trainer/Trainer.cpp

@@ -205,7 +205,7 @@ void Trainer::init(const std::shared_ptr<TrainerConfigHelper>& config,
       (!IGradientMachineMode::dataMustInCpu(mode_, FLAGS_trainer_count));
 
   dataProvider_ = dataProvider;
-  if (!dataProvider_ && config_->hasDataConfig()) {
+  if (!dataProvider_ && config_->hasDataConfig() && !testing_) {
     dataProvider_.reset(DataProvider::create(*config_, *config_, gpuData));
   }
   if (!testDataProvider_) {