-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Add maximum chunk size to data lake store docs. #4464
Conversation
If you're a MSFT employee, click this link |
Can one of the admins verify this patch? |
Automation for azure-sdk-for-jsNothing to generate for azure-sdk-for-js |
Automation for azure-sdk-for-pythonNothing to generate for azure-sdk-for-python |
Automation for azure-sdk-for-rubyNothing to generate for azure-sdk-for-ruby |
Automation for azure-sdk-for-nodeNothing to generate for azure-sdk-for-node |
Automation for azure-sdk-for-javaNothing to generate for azure-sdk-for-java |
Automation for azure-sdk-for-goNothing to generate for azure-sdk-for-go |
...datalake-store/data-plane/Microsoft.DataLakeStore/preview/2015-10-01-preview/filesystem.json
Show resolved
Hide resolved
...datalake-store/data-plane/Microsoft.DataLakeStore/preview/2015-10-01-preview/filesystem.json
Show resolved
Hide resolved
...datalake-store/data-plane/Microsoft.DataLakeStore/preview/2015-10-01-preview/filesystem.json
Show resolved
Hide resolved
} | ||
}, | ||
"x-ms-paths": { | ||
"/webhdfs/v1/{destinationPath}?op=CONCAT": { |
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 there a process to deprecate this Concat. We should only have MSConcat. Concat has some problem for which customer faces issues.
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.
Yes we can add deprecated: true
and update the description string. You want me to do that?
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.
Yes I think we should do that. Concat takes in input as queryparameters which causes another IIS filtering 404 error which is notoriously difficult to catch.
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.
Done, see latest commit.
@lewu-msft Can you also please take a look at this? |
LGTM |
@AutorestCI regenerate azure-sdk-for-go |
* Revert "Add maximum chunk size to data lake store docs. (#4464)" This reverts commit ab9c6ff. * Revert "Add latest authorization tag to java config (#4406)" This reverts commit 7f0fd16. * Revert "Mdb 20181029 statepropdescriptions (#4452)" This reverts commit 90db24c. * Revert "fixing invalid location in examples (#4415)" This reverts commit ffaf595. * Revert "/identity endpoint added (#4460)" This reverts commit 4f93dee. * Revert "Added missing PATCH operation to Api Issue resource (#4440)" This reverts commit 7df0ce5. * Revert "Adds storage account quick failover API (#4457)" This reverts commit 10ec74f. * Revert "DataFactory: add table name in dataset (#4424)" This reverts commit f8465de. * Revert "Update README.md (#4421)" This reverts commit fa6a584. * Revert "Move Ruby SDK config to its own config file (#4451)" This reverts commit 2ca50c8. * Revert "[CognitiveServices] enable Bing Search/CustomSearch products to specify Endpoint (#4401)" This reverts commit dc782b3. * Revert "Add GeoRegion property to DeletedSite model (#4448)" This reverts commit a079367. * Revert " Azure Container Instance: Add start container group API (#4423)" This reverts commit 2ea3e9e. * Revert "add guidance for example names (#4432)" This reverts commit 6780e8a. * Revert "Remove security contact phone field mandatory fields (#4417)" This reverts commit eaf464e. * Revert "Add Allowed Connections REST API (#4106)" This reverts commit 2389da3. * Revert "Make NIC VM readOnly (#4444)" This reverts commit 2d83450. * Revert "Remove readonly attribute from identities collection in users (#4442)" This reverts commit 7577df9. * Revert "Add spec for alert repair/remediate (#4084)" This reverts commit 3f568b4. * Revert "Py conf in its own file (#4439)" This reverts commit 3373760. * Revert "Split Python conf in it's own file (#4438)" This reverts commit 77cf607. * Revert "Adding ManagedInstanceVulnerabilityAssessments and ServerVulnerabilityAssessments (#4416)" This reverts commit dec7549. * Revert "WebApps: Snapshots API for DR stamps (#4291)" This reverts commit 916ff2e. * Revert "Swagger closure for Azure Migration Hub Operations (#4330)" This reverts commit badd1de.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.
Contribution checklist: