Add caching for Table metadata fetch in CDP and update FieldType to Scaler for Salesforce#3016
Merged
jas-valgotar merged 6 commits intomainfrom Feb 6, 2026
Merged
Conversation
Contributor
Author
|
✅ No public API change. |
jas-valgotar
force-pushed
the
jas/addCachingForCDPMetadata
branch
from
8fa6a15 to
0404099
Compare
jas-valgotar
changed the title
Contributor
Author
|
✅ No public API change. |
jas-valgotar
force-pushed
the
jas/addCachingForCDPMetadata
branch
from
43484e0 to
d9fc3f5
Compare
Contributor
Author
|
✅ No public API change. |
jas-valgotar
force-pushed
the
jas/addCachingForCDPMetadata
branch
from
d9fc3f5 to
b6f6389
Compare
Contributor
Author
|
✅ No public API change. |
MikeStall
approved these changes
Feb 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces a new caching mechanism for table metadata in the CDP connector layer to improve performance and reduce redundant network calls. It also updates documentation and corrects a field type resolution behavior. The most significant changes are the introduction of a thread-safe cache for table metadata, propagation of this cache through the relevant classes, and a documentation addition for Claude Code.
Caching and Performance Improvements:
ConcurrentDictionarycache for table metadata inCdpDataSource,CdpTable, andCdpTableResolverto avoid redundant network requests when resolving table schemas. The cache uses the metadata URI as a key and stores tasks for async deduplication. It is cleared when reaching a maximum size to prevent unbounded growth. (src/libraries/Microsoft.PowerFx.Connectors/Tabular/Services/CdpDataSource.cs,src/libraries/Microsoft.PowerFx.Connectors/Tabular/Services/CdpTable.cs,src/libraries/Microsoft.PowerFx.Connectors/Tabular/CdpTableResolver.cs) [1] [2] [3] [4]CdpTableandCdpTableResolverto accept and propagate the cache instance, ensuring all table metadata resolutions use the shared cache. (src/libraries/Microsoft.PowerFx.Connectors/Tabular/Services/CdpDataSource.cs,src/libraries/Microsoft.PowerFx.Connectors/Tabular/Services/CdpTable.cs,src/libraries/Microsoft.PowerFx.Connectors/Tabular/CdpTableResolver.cs) [1] [2] [3] [4]CdpDataSourcefor scenarios where a refresh is required.Documentation:
CLAUDE.mdfile to guide Claude Code (claude.ai/code) on building, testing, and understanding the architecture of the Power Fx codebase. This includes build commands, core architecture, testing, extensibility, and configuration details. (src/CLAUDE.md)Bug Fixes and Behavior Changes:
TryGetFieldTypeinCdpRecordType.csto ignore relationships by default (passingtruefor theignorelationshipparameter), aligning it with the behavior ofTryGetUnderlyingFieldType. (src/libraries/Microsoft.PowerFx.Connectors/Public/CdpRecordType.cs)