[CP] fix zlib error message
This commit is contained in:
		@ -54,11 +54,15 @@ const char *const all_compressor_name[] =
 | 
				
			|||||||
};
 | 
					};
 | 
				
			||||||
 | 
					
 | 
				
			||||||
STATIC_ASSERT(ARRAYSIZEOF(all_compressor_name) == ObCompressorType::MAX_COMPRESSOR, "compressor count mismatch");
 | 
					STATIC_ASSERT(ARRAYSIZEOF(all_compressor_name) == ObCompressorType::MAX_COMPRESSOR, "compressor count mismatch");
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					#define DISABLED_ZLIB_1_COMPRESS_IDX 3
 | 
				
			||||||
 | 
					
 | 
				
			||||||
const char *const compress_funcs[] =
 | 
					const char *const compress_funcs[] =
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
  "lz4_1.0",
 | 
					  "lz4_1.0",
 | 
				
			||||||
  "none",
 | 
					  "none",
 | 
				
			||||||
  "snappy_1.0",
 | 
					  "snappy_1.0",
 | 
				
			||||||
 | 
					  "zlib_1.0", // temporarily disable zlib_1.0
 | 
				
			||||||
  "zstd_1.0",
 | 
					  "zstd_1.0",
 | 
				
			||||||
  "zstd_1.3.8",
 | 
					  "zstd_1.3.8",
 | 
				
			||||||
  "lz4_1.9.1",
 | 
					  "lz4_1.9.1",
 | 
				
			||||||
 | 
				
			|||||||
@ -307,10 +307,13 @@ bool ObConfigStaleTimeChecker::check(const ObConfigItem &t) const
 | 
				
			|||||||
bool ObConfigCompressFuncChecker::check(const ObConfigItem &t) const
 | 
					bool ObConfigCompressFuncChecker::check(const ObConfigItem &t) const
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
  bool is_valid = false;
 | 
					  bool is_valid = false;
 | 
				
			||||||
  for (int i = 0; i < ARRAYSIZEOF(common::compress_funcs) && !is_valid; ++i) {
 | 
					  for (int i = 0; i < ARRAYSIZEOF(common::compress_funcs); ++i) {
 | 
				
			||||||
    if (0 == ObString::make_string(compress_funcs[i]).case_compare(t.str())) {
 | 
					    if (0 == ObString::make_string(compress_funcs[i]).case_compare(t.str())) {
 | 
				
			||||||
 | 
					      if (i != DISABLED_ZLIB_1_COMPRESS_IDX) {
 | 
				
			||||||
        is_valid = true;
 | 
					        is_valid = true;
 | 
				
			||||||
      }
 | 
					      }
 | 
				
			||||||
 | 
					      break;
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
  return is_valid;
 | 
					  return is_valid;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
				
			|||||||
@ -147,7 +147,7 @@ DEF_STR_WITH_CHECKER(_publish_schema_mode, OB_TENANT_PARAMETER, "BEST_EFFORT",
 | 
				
			|||||||
DEF_STR_WITH_CHECKER(default_compress_func, OB_CLUSTER_PARAMETER, "zstd_1.3.8",
 | 
					DEF_STR_WITH_CHECKER(default_compress_func, OB_CLUSTER_PARAMETER, "zstd_1.3.8",
 | 
				
			||||||
                     common::ObConfigCompressFuncChecker,
 | 
					                     common::ObConfigCompressFuncChecker,
 | 
				
			||||||
                     "default compress function name for create new table, "
 | 
					                     "default compress function name for create new table, "
 | 
				
			||||||
                     "values: none, lz4_1.0, snappy_1.0, zlib_1.0, zstd_1.0, zstd_1.3.8",
 | 
					                     "values: none, lz4_1.0, snappy_1.0, zstd_1.0, zstd_1.3.8",
 | 
				
			||||||
                     ObParameterAttr(Section::OBSERVER, Source::DEFAULT, EditLevel::DYNAMIC_EFFECTIVE));
 | 
					                     ObParameterAttr(Section::OBSERVER, Source::DEFAULT, EditLevel::DYNAMIC_EFFECTIVE));
 | 
				
			||||||
 | 
					
 | 
				
			||||||
DEF_STR_WITH_CHECKER(default_row_format, OB_CLUSTER_PARAMETER, "dynamic",
 | 
					DEF_STR_WITH_CHECKER(default_row_format, OB_CLUSTER_PARAMETER, "dynamic",
 | 
				
			||||||
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user