Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quartalgenerierung schlägt fehl #1301

Open
2 tasks
peggimann opened this issue Jan 14, 2025 · 3 comments · May be fixed by #1311
Open
2 tasks

Quartalgenerierung schlägt fehl #1301

peggimann opened this issue Jan 14, 2025 · 3 comments · May be fixed by #1311
Assignees
Labels
bug Something isn't working

Comments

@peggimann
Copy link
Collaborator

Beim OKR Tool sollte immer ein QUartal im Voraus erstellt werden. Beim wechsel auf das neue Quartal ist dies nicht geschehen.

Anforderungen

  • Es werden 6 Quarter Angezeigt, eines im Voraus , das Aktuelle, 4 in die Vergangenheit

Akzeptanzkriterien

  • Der Bug ist behoben
  • Es existiert ein Test, welcher dies sicherstellt.
@peggimann peggimann added the bug Something isn't working label Jan 14, 2025
@peggimann peggimann assigned peggimann and unassigned peggimann Jan 14, 2025
@RandomTannenbaum RandomTannenbaum self-assigned this Jan 15, 2025
@RandomTannenbaum
Copy link
Collaborator

Stand 15.01.2025
Ich habe die scheduled task im QuarterBusinessService angeschaut. Diese scheint soweit richtig zu sein, jedoch bekomme ich in der Konsole einen Fehler, wenn der service versucht das Quarter zu speichern, der lautet: ERROR: relation "sequence_quarter" does not exist.

Tipp: Intellij erlaubt es per knopfdruck eine cron-expression sofort auszuführen.

@Miguel7373 Miguel7373 self-assigned this Jan 16, 2025
@Miguel7373
Copy link
Collaborator

Miguel7373 commented Jan 16, 2025

Stand 16.01.2025

Ich habe mir genauer angesehen, wo der Fehler sich befindet. Das Problem liegt daran, dass es versucht, im falschen Schema zu holen, da im, SchemaMultiTenantConnectionProvider da wir noch keinen Tendant haben, einfach public als Schema aus dem TendantContext geholt wird.

Ich habe die Applikation so umgeändert, dass alle 30 Sekunden nach dem Starten die Scheduled Quarter Generation gestartet wird.

@MasterEvarior
Copy link
Collaborator

MasterEvarior commented Jan 17, 2025

Stand 17.01.2025

I've added a crude fix, which simply gets all the available schemas and iterates over them. This seems to work okay. This can certainly be written a bit better though.

Explanation of the bug

The tenant only gets set on the current thread. Each method that is annotated with @Schedule gets executed in a separate Thread. This leads to the schema always being public.

TODOs

  • Make fix less crude
  • Fix existing tests
  • Add tests(s)

@MasterEvarior MasterEvarior linked a pull request Jan 17, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants