Don't use non empty domain on empty array.

This was introduced in #4710. In that PR, we could try to use the non empty domain of an array with no fragments, which is empty. This should fix python and R nightlies.

---
TYPE: NO_HISTORY
DESC: Don't use non empty domain on empty array.

tiledb/sm/query/readers/sparse_global_order_reader.cc

@@ -119,8 +119,6 @@ Status SparseGlobalOrderReader<BitmapType>::dowork() {
   auto timer_se = stats_->start_timer("dowork");
   stats_->add_counter("loop_num", 1);
 
-  subarray_.reset_default_ranges();
-
   // Check that the query condition is valid.
   if (condition_.has_value()) {
     throw_if_not_ok(condition_->check(array_schema_));
@@ -137,6 +135,8 @@ Status SparseGlobalOrderReader<BitmapType>::dowork() {
     return Status::Ok();
   }
 
+  subarray_.reset_default_ranges();
+
   // Load initial data, if not loaded already.
   throw_if_not_ok(load_initial_data());
   purge_deletes_consolidation_ = !deletes_consolidation_no_purge_ &&

tiledb/sm/query/readers/sparse_unordered_with_dups_reader.cc

@@ -125,10 +125,6 @@ Status SparseUnorderedWithDupsReader<BitmapType>::dowork() {
   auto timer_se = stats_->start_timer("dowork");
   stats_->add_counter("loop_num", 1);
 
-  // Subarray is not known to be explicitly set until buffers are deserialized
-  subarray_.reset_default_ranges();
-  include_coords_ = subarray_.is_set();
-
   // Make sure user didn't request delete timestamps.
   if (buffers_.count(constants::delete_timestamps) != 0) {
     return logger_->status(Status_SparseUnorderedWithDupsReaderError(
@@ -154,6 +150,10 @@ Status SparseUnorderedWithDupsReader<BitmapType>::dowork() {
     return Status::Ok();
   }
 
+  // Subarray is not known to be explicitly set until buffers are deserialized
+  subarray_.reset_default_ranges();
+  include_coords_ = subarray_.is_set();
+
   // Load initial data, if not loaded already. Coords are only included if the
   // subarray is set.
   throw_if_not_ok(load_initial_data());