Skip to content

Conversation

@nmalkapuram
Copy link
Contributor

No description provided.

bool isX64 = this.CpuArchitecture == System.Runtime.InteropServices.Architecture.X64;
bool isArm64 = this.CpuArchitecture == System.Runtime.InteropServices.Architecture.Arm64;

bool isArchitectureSupported = isX64 || (isServerExecutorInServerRole && isArm64);
Copy link
Contributor

Choose a reason for hiding this comment

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

Please document which distro is supported on which arch in the document.

"MinimumExecutionInterval": "00:01:00",
"Metadata": {
"RecommendedMinimumExecutionTime": "04:00:00",
"SupportedPlatforms": "linux-x64",
Copy link
Contributor

Choose a reason for hiding this comment

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

Profile says ARM64 but here it says linux-x64, first, is it possible that we run on both x64 and arm64. Second could we merge the profile?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was redundant profile, I have added the working profile to blob storage. That works for asymmetric client server architecture. (arm46 server and x64 client). Let me know if I need to add this here.

Copy link
Contributor

Choose a reason for hiding this comment

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

For discoverability please add here and use arch agnostic profile if available.

Copy link
Contributor

@yangpanMS yangpanMS left a comment

Choose a reason for hiding this comment

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

Please update documents and verify the supported distro and architecture

@nmalkapuram nmalkapuram force-pushed the users/nmalkapuram/postresqlhammerdbsupportonARM branch from 889af85 to da38bf5 Compare September 19, 2025 04:32
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.

3 participants