-
Notifications
You must be signed in to change notification settings - Fork 723
fix: compatibility notice for sagemaker notebook al2-v3 #3243
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
Hey @ericfzhu thanks for opening this. It is true that from pyarrow v21+ it stopped building on AL2 due to an older CMake, but you should still be able to use an older version pyarrow (technically The notice makes me think we require pyarrow 21+ which isn't exactly true, perhaps it needs to be slightly updated. |
|
Would pinning or installing a lower version of PyArrow be a better recommendation here? Basically something like:
|
|
@ericfzhu Something along the lines of:
|
…mpatibility) and possible solutions
|
This should better address what the actual problem is, and potential solutions to address the issue |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
Thanks @ericfzhu ! |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
A potential solution to address #3242 for users with AL2-V3 SageMaker notebook instances.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.