Kafka
  1. Kafka
  2. KAFKA-1084

Validate properties for custom serializers

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.8.1
    • Component/s: None
    • Labels:
      None

      Description

      We use specifics encoder/decoder for our producers/consumers, they get correctly initialized by the Producer/Consumer. The only downside is the validate() function of VerifiableProperties that pollutes our log stream.

      This patch allows custom serializers keys to validate correctly if they begin with the "external" prefix, for example:

      external.my.encoder.param=true

      will not raise a WARN.

        Activity

        Hide
        Jun Rao added a comment -

        Thanks for the patch. Do you think the patch in KAFKA-1049 addresses the problem better?

        Show
        Jun Rao added a comment - Thanks for the patch. Do you think the patch in KAFKA-1049 addresses the problem better?
        Hide
        Francois Saint-Jacques added a comment -

        I believe a warning will still get logged for unknown string properties since the patch in KAFKA-1049 doesn't touch the validate function.

        Show
        Francois Saint-Jacques added a comment - I believe a warning will still get logged for unknown string properties since the patch in KAFKA-1049 doesn't touch the validate function.
        Hide
        Jun Rao added a comment -

        Yes, you are right. +1 on the patch. Committed to trunk.

        Show
        Jun Rao added a comment - Yes, you are right. +1 on the patch. Committed to trunk.

          People

          • Assignee:
            Francois Saint-Jacques
            Reporter:
            Francois Saint-Jacques
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development