performance regression in JsonSchemaConverter between 1.51.0 and 1.52.0

[ghostbuster91]

Hi,

While updating from 1.51.0 to 1.52.0 we noticed that one of our operation started to take significantly more time (approximately 10 times more). After some debugging and profiling it turned out to be related to the recent change in JsonSchemaConverter.

We worked it around by minimizing the model to only relevant shapes before passing it to JsonSchemaConverter. Previously we were passing there the full model which is quite big (around 12k shapes).

Maybe there are some optimizations that could be applied to scrubTraitDefinitions to make it faster.