Skip to content

[sync] T4086 Avoid computed updates through unavailable tables T4086#3219

Closed
tea-artist wants to merge 1 commit into
developfrom
sync/ee-20260522-021555
Closed

[sync] T4086 Avoid computed updates through unavailable tables T4086#3219
tea-artist wants to merge 1 commit into
developfrom
sync/ee-20260522-021555

Conversation

@tea-artist
Copy link
Copy Markdown
Contributor

@tea-artist tea-artist commented May 22, 2026

🔄 Automated sync from EE repository.

49 commit(s) synced since last sync.

Authors

Included commits

  • T4086 Avoid computed updates through unavailable tables T4086 (nichenqin)
  • fix(app): handle lookup field editor null options (nichenqin)
  • fix(v2): optimize conditional lookup field reference query (nichenqin)
  • fix: keep tables ready after update failures T4661 (nichenqin)
  • fix(selection-statistic): clarify copy feedback T4616 (Jun Lu)
  • fix: improve user filter dropdown T4390 (Jun Lu)
  • fix(T4609): add form view side gutter (Jun Lu)
  • feat(cross-space): forbid cross-space link/lookup/rollup across create, duplicate, move (T3837) (Aries X)
  • feat/automation history new layout (Jun Lu)
  • fix: chat queue force send render (Boris)
  • Add v2 full rollout admin workflow T3836 (nichenqin)
  • Enhance AI agent task progress and runtime context T4093 T4658 (Pengap)
  • T4357 fix: preserve computed filter before images (nichenqin)
  • feat(cli): base CRUD + command toggle + private-deploy (T4597, T4606) (Boris)
  • T4357 fix: reduce computed update storm (nichenqin)
  • fix(sdk): refresh grouped record queries after mutations (nichenqin)
  • fix(toolbar): prevent overlap on narrow widths T4148 (Jun Lu)
  • fix(t4595): enforce space owner limit only on free spaces (Uno)
  • T4594 Fix BYODB AI field generation without CLS context (nichenqin)
  • T4598 fix stale grouped grid cache (nichenqin)
  • fix(view): drop submit.allow flag, gate form-submit on view.type (T4612) (Aries X)
  • fix(v2): defer computed tasks for inactive tables (nichenqin)
  • fix(sdk): reconcile query changes for grid records (nichenqin)
  • fix(v2): fall back to async computed backfill (nichenqin)
  • fix(app): avoid redirecting to unavailable table nodes (nichenqin)
  • feat(T4591): update changelog banner (Jocky-Teable)
  • fix(v2): resolve fk repair targets from table meta (nichenqin)
  • T3769 Add v2 duplicate base streaming (nichenqin)
  • fix(v2): reduce record order query parameters T4456 (nichenqin)
  • feat(app): support existing space BYODB binding T4162 (nichenqin)
  • fix(app): share view redirected to login on open (T4490) (Aries X)
  • fix: chunk wide computed updates (nichenqin)
  • fix: optimize conditional rollup computed queries (nichenqin)
  • fix(v2): preserve table references on soft delete (nichenqin)
  • Enhance IM chat context understanding T4258 (Pengap)
  • chore: update system prompt T4192 (Bieber)
  • feat(backend): support v2 tea export T3699 (nichenqin)
  • fix(v2): enforce select choice name limits (nichenqin)
  • fix: repair missing-column v2 schema operations (nichenqin)
  • fix: persist chat input draft per chatId across refresh (T4142) (Aries X)
  • fix(core): normalize numeric formula field values (nichenqin)
  • feat: add BYODB space data database support T3766 (nichenqin)
  • feat(t4133): add admin notification feature with validation and service integration (Uno)
  • fix: scenario error cleanup consistency (Boris)
  • fix(v2): allow delete with duplicate linked field names (nichenqin)
  • fix: enforce v2 table data safety limits (T4130) (nichenqin)
  • fix: handle blank number formula output (T4128) (nichenqin)
  • feat(t4094): update changelog banner to @ Any Node in AI Chat (Jocky-Teable)
  • Improve AI generation flow reliability T4091 T4084 (Pengap)

Latest source commit: teableio/teable-ee@99a5c55

This PR was automatically created by the sync workflow.

@github-actions @caoxing9
@tea-artist @boris-w @Jocky-Teable @hammond-lj @Pengap @younocode @nichenqin
[sync] T4086 Avoid computed updates through unavailable tables T4086
ef3554d 
Synced from teableio/teable-ee@99a5c55

Co-authored-by: Aries X <caoxing9@gmail.com>
Co-authored-by: Bieber <artist@teable.io>
Co-authored-by: Boris <boris2code@outlook.com>
Co-authored-by: Jocky-Teable <jocky@teable.ai>
Co-authored-by: Jun Lu <hammond@teable.io>
Co-authored-by: Pengap <penganpingprivte@gmail.com>
Co-authored-by: Uno <uno@teable.ai>
Co-authored-by: nichenqin <nichenqin@hotmail.com>
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 22, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed May 22, 2026
View reviewed changes
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ef3554d956

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/nestjs-backend/src/distributed-lock/distributed-lock.service.ts
Comment on lines +76 to +77
if ((await this.cacheService.get(key)) === this.owner) {
await this.cacheService.del(key);
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot May 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Release Redis lock atomically with owner check

The get + del sequence is not atomic, so a lock can be deleted after ownership changes: if this process reads its own owner value, then the key expires and another instance acquires it before del runs, the subsequent del removes the new owner’s lock. That breaks mutual exclusion and can allow overlapping critical sections when tasks run near TTL boundaries. Use an atomic compare-and-delete (e.g., a Lua script) instead of separate calls.

Useful? React with 👍 / 👎.

@tea-artist tea-artist closed this May 22, 2026
@tea-artist tea-artist deleted the sync/ee-20260522-021555 branch May 22, 2026 08:00
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 22, 2026

🧹 Preview Environment Cleanup

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tea-artist @CLAassistant