[CAMEL-19010] file component: confirm not called on idempotentRepository

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.20.1
• Fix Version/s: 4.7.0
• Component/s: camel-file
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: Marco Bungart
• Created: 03/Feb/23 22:59
• Updated: 16/Jun/24 11:03
• Resolved: 16/Jun/24 11:03

Description

When we bind an idempotent repository to a file component:

from(
    file("in")
        .noop(true)
        .idempotent(true)
        .idempotentRepository(idempotentRepository)
        .idempotentKey("${file:name}"))

    .

    .

    .

then method confirm(...) should be called after the exchange has been processed.

—

Expected behaviour: method confirm(...) should be called after the exchange has been processed

—

Actual behaviour: method confirm(...) is not called.

—

Reproducer:

• Working scenario: bind the idempotency repository "after" the file producer:
  • checkout this github.com repo
  • start the database: cd local-deployment && docker compose up -d && cd ..
  • start the quarkus application. ./mvnw quarkus:dev
  • in a separate terminal, move some file in folder in
  • go to pgadmin4: http://localhost:8091 (username: [e-mail], password: pgadmin4)
  • on the left side, click on docker, then on postgres. When prompted for a password for databaes postgres, enter postgres
  • click Databases -> camel -> Schemas -> public -> tables, right-click idempotency, select "show data"
  • observe that the entry has the column done set to true
  • stop the quarkus application
  • clear database table idempotency
  • delete folders in and out: rm -rf in && rm -rf out
• Bug scenario:
  • switch to branch repository-on-file-producer: git fetch && git switch repository-on-file-producer
  • start quarkus application and repeat the test
  • go to pgadmin4, observe that a row was created, but done is not set to true

Going further, we can also set a break point in or on method confirm(...), and it will not be triggered.

—

Remark: this holds also true when a repository is set as inProgressRepository(...)

—

FTR: Relevant zulip chat