Uploaded image for project: 'Debezium'
  1. Debezium
  2. DBZ-4191

wrong variable naming in an unit test for Outbox Event Router SMT

    XMLWordPrintable

Details

    • Enhancement
    • Resolution: Done
    • Optional
    • 1.8.0.Alpha1
    • None
    • core-library
    • None
    • False
    • False

    Description

      following code is from EventRouterTest.java

      Envelope envelope = Envelope.defineSchema()
              .withName("event.Envelope")
              .withRecord(recordSchema)
              .withSource(SchemaBuilder.struct().build())
              .build();
      
      final Struct before = new Struct(recordSchema);
      before.put("id", eventId);
      before.put("aggregatetype", payloadType);
      before.put("aggregateid", payloadId);
      before.put("type", eventType);
      before.put("payload", payload);
      
      extraValues.forEach(before::put);
      
      final Struct body = envelope.create(before, null, Instant.now());
      

      Actually, 'before' should be 'after' because there is no 'before' field in 'create' change event. I think it should be fixed so that the test could help another developers, such as new comer like me, who read this test to try to understand how MySQL Connector works.

      Attachments

        Activity

          People

            Unassigned Unassigned
            sunghohwang Sungho Hwang (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: