Uploaded image for project: 'Samza'
  1. Samza
  2. SAMZA-473

Remove samza-serializers package

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 0.9.0
    • 0.9.0
    • container

    Description

      The samza-serializers package should be removed, and its contents moved back into samza-core. Originally, the serializers package contained a JSON serde, and was isolated from samza-core to avoid introducing a Jackson dependency. However, Jackson was introduced as a dependency anyway. In addition, other serializers (e.g. an avro serializer, or a Protobuf serializer) should not reside in the same package (samza-serializers), but in their own packages (samza-avro, samza-protobuf) to avoid contaminating dependencies.

      In conclusion, this package is pointless, and should be removed.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            criccomini Chris Riccomini
            criccomini Chris Riccomini
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment