Update Jackson to 2.18.4

[piotrp]

Describe your changes

TODO: solve Jackson incompatibility by removing changes from #7066. To sum up Slack discussion:

If it's unused let's remove it

Checklist before merge

• Related issue ID is placed at the beginning of PR title in [brackets] (can be GH issue or Nu Jira issue)
• Code is cleaned from temporary changes and commented out lines
• Parts of the code that are not easy to understand are documented in the code
• Changes are covered by automated tests
• Showcase in dev-application.conf added to demonstrate the feature
• Documentation added or updated
• Added entry in Changelog.md describing the change from the perspective of a public distribution user
• Added MigrationGuide.md entry in the appropriate subcategory if introducing a breaking change
• Verify that PR will be squashed during merge

[lciolecki]

Hmm, do we even need these two schemas?

[piotrp]

Unfortunately, yes:

class UniversalSchemaSupportDispatcher private (kafkaConfig: KafkaConfig) {
...
  def forParsedSchema(parsedSchema: ParsedSchema): UniversalSchemaSupport = parsedSchema match {
    // For ad hoc tests we want to present the user with json editor when topic has no schema and content type Json was selected
    case ContentTypesSchemas.schemaForJson => NoSchemaJsonSupport
    case _                                 => forSchemaType(parsedSchema.schemaType())
  }

[lciolecki]

Are you sure about this? :)

[piotrp]

Yes, it's that or some heavier refactoring. true after parsing results in the same schema: {}. An alternative would be to use {} ({} with some whitespace around) with custom equals method for OpenAPIJsonSchema.

I mention custom equals because OpenAPIJsonSchema is a fake case class - default Scala's equals for case classes doesn't work there, it's implemented like this:

  test("boxes are equal") {
    class BoxParent {
      override def equals(obj: Any): Boolean = true
      override def hashCode(): Int = 0
    }

    case class Box(v: String) extends BoxParent

    // this equality is true in Scala, but not in real world
    Box("socks") shouldBe Box("gold")
  }