Enable bool compression

This change sets the default for the GUC value:

`timescaledb.enable_bool_compression=true`

Author: dbeck

.unreleased/pr_8014

@@ -0,0 +1 @@
+Implements: #8014 enable bool compression by default by setting `timescaledb.enable_bool_compression=true`

src/guc.c

@@ -154,7 +154,7 @@ TSDLLEXPORT bool ts_guc_default_hypercore_use_access_method = false;
 bool ts_guc_enable_chunk_skipping = false;
 TSDLLEXPORT bool ts_guc_enable_segmentwise_recompression = true;
 TSDLLEXPORT bool ts_guc_enable_exclusive_locking_recompression = false;
-TSDLLEXPORT bool ts_guc_enable_bool_compression = false;
+TSDLLEXPORT bool ts_guc_enable_bool_compression = true;
 TSDLLEXPORT int ts_guc_compression_batch_size_limit = 1000;
 
 /* Only settable in debug mode for testing */
@@ -795,10 +795,10 @@ _guc_init(void)
 							 NULL);
 
 	DefineCustomBoolVariable(MAKE_EXTOPTION("enable_bool_compression"),
-							 "Enable experimental bool compression functionality",
+							 "Enable bool compression functionality",
 							 "Enable bool compression",
 							 &ts_guc_enable_bool_compression,
-							 false,
+							 true,
 							 PGC_USERSET,
 							 0,
 							 NULL,

tsl/src/nodes/decompress_chunk/compressed_batch.c

@@ -221,18 +221,25 @@ decompress_column(DecompressContext *dcontext, DecompressBatchState *batch_state
 		DecompressAllFunction decompress_all =
 			tsl_get_decompress_all_function(header->compression_algorithm,
 											column_description->typid);
-		Assert(decompress_all != NULL);
 
-		MemoryContext context_before_decompression =
-			MemoryContextSwitchTo(dcontext->bulk_decompression_context);
+		/*
+		 * The decompress_all function is not defined for ARRAY and DICTIONARY
+		 * compression types, except for TEXTOID, so it is valid to have
+		 * a NULL pointer here, and it must be handled.
+		 */
+		if (decompress_all != NULL)
+		{
+			MemoryContext context_before_decompression =
+				MemoryContextSwitchTo(dcontext->bulk_decompression_context);
 
-		arrow = decompress_all(PointerGetDatum(header),
-							   column_description->typid,
-							   batch_state->per_batch_context);
+			arrow = decompress_all(PointerGetDatum(header),
+								   column_description->typid,
+								   batch_state->per_batch_context);
 
-		MemoryContextSwitchTo(context_before_decompression);
+			MemoryContextSwitchTo(context_before_decompression);
 
-		MemoryContextReset(dcontext->bulk_decompression_context);
+			MemoryContextReset(dcontext->bulk_decompression_context);
+		}
 	}
 
 	if (arrow == NULL)