fix: OpenTelemetry metric names must only contain [A-Za-z0-9_.-]

hub/hub.go

@@ -120,7 +120,7 @@
 	h.telemetryShutdownFunc = shutdownTelemetry
 
 	hydrateCalls, err := otel.GetMeterProvider().Meter(constants.FdwName).Int64Counter(
-		fmt.Sprintf("%s/hydrate_calls_total", constants.FdwName),
+		fmt.Sprintf("%s-hydrate_calls_total", constants.FdwName),
 		metric.WithDescription("The total number of hydrate calls"),
 	)
 	if err != nil {
@@ -269,7 +269,7 @@
 // GetIterator creates and returns an iterator
 func (h *Hub) GetIterator(columns []string, quals *proto.Quals, unhandledRestrictions int, limit int64, opts types.Options) (Iterator, error) {
 	logging.LogTime("GetIterator start")
 	qualMap, err := h.buildQualMap(quals)
 	connectionName := opts["connection"]
 	table := opts["table"]
 	log.Printf("[TRACE] Hub GetIterator() table '%s'", table)
@@ -397,9 +397,9 @@
 	// build path keys based on the table key columns
 	// NOTE: the schema data has changed in SDK version 1.3 - we must handle plugins using legacy sdk explicitly
 	// check for legacy sdk versions
 	if tableSchema.ListCallKeyColumns != nil {
 		log.Printf("[TRACE] schema response include ListCallKeyColumns, it is using legacy protobuff interface ")
 		pathKeys = types.LegacyKeyColumnsToPathKeys(tableSchema.ListCallKeyColumns, tableSchema.ListCallOptionalKeyColumns, allColumns)
 	} else if tableSchema.ListCallKeyColumnList != nil {
 		log.Printf("[TRACE] schema response include ListCallKeyColumnList, it is using the updated protobuff interface ")
 		// generate path keys if there are required list key columns
@@ -776,12 +776,12 @@
 	switch command {
 	case constants.LegacyCommandCacheClear:
 		// set the cache clear time for the remote query cache
 		h.cacheSettings.Apply(string(settings.SettingKeyCacheClearTimeOverride), "")
 
 	case constants.LegacyCommandCacheOn:
 		h.cacheSettings.Apply(string(settings.SettingKeyCacheEnabled), "true")
 	case constants.LegacyCommandCacheOff:
 		h.cacheSettings.Apply(string(settings.SettingKeyCacheClearTimeOverride), "false")
 	}
 	return nil
 }