-
Notifications
You must be signed in to change notification settings - Fork 4.9k
update generated code, add connections + indexes #49476
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
base: feature/azure-ai-projects-1dp
Are you sure you want to change the base?
update generated code, add connections + indexes #49476
Conversation
sophia-ramsey
commented
Apr 17, 2025
- update generated code based on commit: 1a7c4e8c06b6727eec8fe97795e0c990e8910044
- add samples folder for Datasets sample
- add connections and indexes samples (still waiting for service side functionality)
…ure/azure-sdk-for-net into sramsey/datasets-deployments-onedp
API change check APIView has identified API level changes in this PR and created following API reviews. |
{ | ||
throw new InvalidOperationException("Invalid blob reference for consumption."); | ||
} | ||
//if (pendingUploadResponse.Value.BlobReferenceForConsumption == null || |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is commented code needed?
using System.Collections.Generic; | ||
using System.Linq; | ||
//using System; | ||
//using System.Collections.Generic; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here also commented code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check all places
@@ -0,0 +1 @@ | |||
This is sample file 5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need so many sample_file1-8.txt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sample is testing file uploads for nested folders, since we had to write custom upload file and upload folder functions.
} | ||
} | ||
} | ||
//#nullable disable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We shall remove Agents custom code from https://github.com/Azure/azure-sdk-for-net/tree/feature/azure-ai-projects-1dp, which means you can remove all Agents custom code