Skip to content

Conversation

@joeg
Copy link
Contributor

@joeg joeg commented Mar 18, 2025

Summary: Enable overrides for getPreferredSize behavior on a per memory pool basis

Reviewed By: xiaoxmeng

Differential Revision: D71427371

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 18, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71427371

@netlify
Copy link

netlify bot commented Mar 18, 2025

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit bc177ba
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/67daf4454b3de80008beaff7

@joeg joeg changed the title misc(memory) - Enable preferred memory size override for MemoryPool misc(memory): Enable preferred memory size override for MemoryPool Mar 18, 2025
@joeg joeg force-pushed the export-D71427371 branch from c9f0bda to b8aa1f8 Compare March 19, 2025 16:13
joeg added a commit to joeg/velox that referenced this pull request Mar 19, 2025
…acebookincubator#12712)

Summary:

Enable overrides for getPreferredSize behavior on a per memory pool basis

Reviewed By: xiaoxmeng

Differential Revision: D71427371
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71427371

joeg added a commit to joeg/velox that referenced this pull request Mar 19, 2025
…acebookincubator#12712)

Summary:
Pull Request resolved: facebookincubator#12712

Enable overrides for getPreferredSize behavior on a per memory pool basis

Reviewed By: xiaoxmeng

Differential Revision: D71427371
@joeg joeg force-pushed the export-D71427371 branch from b8aa1f8 to f4a9aad Compare March 19, 2025 16:16
…acebookincubator#12712)

Summary:

Enable overrides for getPreferredSize behavior on a per memory pool basis

Reviewed By: xiaoxmeng

Differential Revision: D71427371
@joeg joeg force-pushed the export-D71427371 branch from f4a9aad to bc177ba Compare March 19, 2025 16:43
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D71427371

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in a34e1b2.

@conbench-facebook
Copy link

Conbench analyzed the 1 benchmark run on commit a34e1b29.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

jinchengchenghh pushed a commit to jinchengchenghh/velox that referenced this pull request Apr 4, 2025
…acebookincubator#12712)

Summary:
Pull Request resolved: facebookincubator#12712

Enable overrides for getPreferredSize behavior on a per memory pool basis

Reviewed By: xiaoxmeng

Differential Revision: D71427371

fbshipit-source-id: ce4e1c9682de7d00af97ca7eb9a6323c4c8ffd02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants