Further MergeTree Client Legacy Deprecations

[anthony-murphy]

In an effort the reduce exposure of the Client class in the merge-tree package this change additionally deprecates a number of types which either directly or indirectly expose the merge-tree Client class.

Most of these types are not meant to be used directly, and direct use is not supported:

• AttributionPolicy
• IClientEvents
• IMergeTreeAttributionOptions
• SharedSegmentSequence
• SharedStringClass

Some of the deprecations are for class constructors and in those cases we plan to replace the class with an interface which has an equivalent API. Direct instantiation of these classes is not currently supported or necessary for any supported scenario, so the change to an interface should not impact usage:

• SequenceInterval
• SequenceEvent
• SequenceDeltaEvent
• SequenceMaintenanceEvent

[tylerbutler]

Changeset looks great - just wording tweaks and minor formatting. Your level of detail was great.

Typically I would suggest trying to link to the docs for the types, but in this case that seems unnecessary since these types aren't likely to be used right now anyway. Maybe links to docs for deprecations in general is unnecessary. Links to docs make way more sense with new or changed APIs. @microsoft/fluid-cr-docs Curious what others think. We can take that discussion to another forum though.

[msfluid-bot]

⯅ @fluid-example/bundle-size-tests: +245 Bytes

Metric Name	Baseline Size	Compare Size	Size Diff
aqueduct.js	461.42 KB	461.45 KB	⯅ +35 Bytes
azureClient.js	559.03 KB	559.08 KB	⯅ +49 Bytes
connectionState.js	680 Bytes	680 Bytes	■ No change
containerRuntime.js	261.68 KB	261.69 KB	⯅ +14 Bytes
fluidFramework.js	404.99 KB	405 KB	⯅ +14 Bytes
loader.js	134.17 KB	134.19 KB	⯅ +14 Bytes
map.js	42.43 KB	42.44 KB	⯅ +7 Bytes
matrix.js	146.8 KB	146.8 KB	⯅ +7 Bytes
odspClient.js	526.18 KB	526.23 KB	⯅ +49 Bytes
odspDriver.js	97.8 KB	97.82 KB	⯅ +21 Bytes
odspPrefetchSnapshot.js	42.76 KB	42.78 KB	⯅ +14 Bytes
sharedString.js	163.79 KB	163.8 KB	⯅ +7 Bytes
sharedTree.js	395.45 KB	395.46 KB	⯅ +7 Bytes
Total Size	3.31 MB	3.31 MB	⯅ +245 Bytes

---

Baseline commit: 7febffe

Generated by 🚫 dangerJS against a3a41dd

[github-actions]

🔗 No broken links found! ✅

Your attention to detail is admirable.

linkcheck output

> [email protected] ci:linkcheck /home/runner/work/FluidFramework/FluidFramework/docs
> start-server-and-test ci:start 1313 linkcheck:full

1: starting server using command "npm run ci:start"
and when url "[ 'http://127.0.0.1:1313' ]" is responding with HTTP status code 200
running tests using command "npm run linkcheck:full"


> [email protected] ci:start
> http-server ./public --port 1313 --silent


> [email protected] linkcheck:full
> npm run linkcheck:fast -- --external


> [email protected] linkcheck:fast
> linkcheck http://localhost:1313 --skip-file skipped-urls.txt --external

Crawling...

Stats:
  409152 links
    3178 destination URLs
       2 URLs ignored
       0 warnings
       0 errors