Merge pull request #1211 from openml/no_pq_on_test

Don't serve parquet from test server as they point to production

Author: PGijsbers

openml_OS/models/api/v1/Api_data.php

@@ -842,7 +842,10 @@ private function data($data_id) {
     if ($data_status != false) {
       $dataset->status = $data_status->status;
     }
-    if ($dataset->format != 'Sparse_ARFF') {
+    // The BASE_URL check prevents servering parquet urls from the test server,
+    // which is needed as long as the test server does not have its own dedicated
+    // MinIO with parquet files. TODO: Remove this after running MinIO on test
+    if ($dataset->format != 'Sparse_ARFF' && BASE_URL != "https://test.openml.org/") {
       $bracket = sprintf('%04d', floor($data_id / 10000));
       $padded_id = sprintf('%04d', $data_id);
       $url = MINIO_URL . 'datasets/' . $bracket . '/' . $padded_id . '/dataset_' . $data_id . '.pq';