Skip to content

Molecule and PS bulk import by file#7651

Open
XingY wants to merge 15 commits into
developfrom
fb_moleculeImport
Open

Molecule and PS bulk import by file#7651
XingY wants to merge 15 commits into
developfrom
fb_moleculeImport

Conversation

@XingY
Copy link
Copy Markdown
Contributor

@XingY XingY commented May 7, 2026

Rationale

Adds a DataClass-specific DataIterator transformer extension point so modules can inject computed columns (e.g., transforming flat import columns into JSON). This handles all import paths uniformly — file upload, API insertRows, folder import, and async import.
Also propagates the useTransactionAuditCache flag through extraScriptContext so nested trigger inserts (component DataClass rows created by MoleculeTriggerFactory) share the outer transaction's audit context.

Related Pull Requests

Changes

  • ExperimentService: registerDataClassDataIteratorTransformer and getDataClassDataIteratorTransformer.
  • ExpDataClassDataTableImpl.PreTriggerDataIteratorBuilder**: Applies the registered transformer so computed columns are available to all downstream stages.
  • ExpDataIterators: useTransactionAuditCache for INSERT operations is now enabled unless explicitly opted out via Boolean.FALSE config parameter (was unconditional). This ensures insert/update called from a trigger matches behavior of outer insert/update.
  • AbstractQueryUpdateService._importRowsUsingDIB: reads useTransactionAuditCache from extraScriptContext and propagates to DataIteratorContext for nested trigger inserts
  • AuditController.GetTransactionRowIds: API response now includes dataTypeRowCounts map alongside rowIds

@XingY XingY requested a review from labkey-nicka May 14, 2026 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant