Allow registering search attributes even when AdvancedVisibility is not enabled

[longquanzheng]

cadence/service/frontend/adminHandler.go

Line 220 in 3b5c2b6

return adh.error(&types.BadRequestError{Message: "AdvancedVisibilityStore is not configured for this Cadence Cluster"}, scope)

This is already supported in Temporal.
With this, search attributes can be used as data fields and not for search.
It's a convenient to test the search attribute API in the workflow code before using advanced visibility.

[lancezhao-ins]

Hi @longquanzheng ,
Does this ticket only require changing the validation around the line you mentioned?

[longquanzheng]

Hi @longquanzheng ,
Does this ticket only require changing the validation around the line you mentioned?

Yes I believe so.

[lancezhao-ins]

Cool, can I pick it up as my first Open-Source ticket?

[longquanzheng]

Cool, can I pick it up as my first Open-Source ticket?

Sure, go ahead. Feel free to ask if you have more questions

[lancezhao-ins]

PR Opened: #5185