[tracker-miners/wip/carlosg/chain-error-propagation] libtracker-miner: Chain up TrackerBatch errors in TrackerSparqlBuffer




commit fdce1f1a982e55583817d6d575cc498fe6f10b34
Author: Carlos Garnacho <carlosg gnome org>
Date:   Fri Jun 25 13:03:42 2021 +0200

    libtracker-miner: Chain up TrackerBatch errors in TrackerSparqlBuffer
    
    In case of batch errors, we are returning early here without finishing
    the GTask. The error should be propagated here, so TrackerMinerFiles
    can report errors and follow up.

 src/libtracker-miner/tracker-sparql-buffer.c | 6 ------
 1 file changed, 6 deletions(-)
---
diff --git a/src/libtracker-miner/tracker-sparql-buffer.c b/src/libtracker-miner/tracker-sparql-buffer.c
index 704e61e9e..814cc1512 100644
--- a/src/libtracker-miner/tracker-sparql-buffer.c
+++ b/src/libtracker-miner/tracker-sparql-buffer.c
@@ -204,12 +204,6 @@ batch_execute_cb (GObject      *object,
        if (!tracker_batch_execute_finish (TRACKER_BATCH (object),
                                           result,
                                           &error)) {
-               g_critical ("Error executing batch: %s\n", error->message);
-               g_error_free (error);
-               return;
-       }
-
-       if (error) {
                g_task_return_error (update_data->async_task, error);
        } else {
                g_task_return_pointer (update_data->async_task,


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]