Skip to content

Fix media streaming bug#8385

Merged
srinathnarayanan merged 8 commits intomasterfrom
srnara/mediafix
Nov 7, 2019
Merged

Fix media streaming bug#8385
srinathnarayanan merged 8 commits intomasterfrom
srnara/mediafix

Conversation

@srinathnarayanan
Copy link
Contributor

@srinathnarayanan srinathnarayanan commented Nov 5, 2019

  • Removed media streaming related checks in synchronized_request that was causing issues with DBAs with "media" in the name
  • Refactored pipeline_client.run into a _PipelineRun method for testing purposes
  • Removed all attachment and media APIs which are unused in the cosmos_client_connection
  • Removed MediaReadmode and MediaRequestTimeout

- Removed media streaming related checks in synchronized_request that was causing issues with DBAs with "media" in the name
- Refactored pipeline_client.run into a _PipelineRun method for testing purposes
@adxsdk6
Copy link

adxsdk6 commented Nov 5, 2019

Can one of the admins verify this patch?

@annatisch annatisch added Client This issue points to a problem in the data-plane of the library. Cosmos labels Nov 5, 2019
@annatisch
Copy link
Member

/azp run python - cosmos - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@srinathnarayanan srinathnarayanan merged commit 03460b3 into master Nov 7, 2019
xiangyan99 added a commit that referenced this pull request Nov 8, 2019
* Updated minor version for released packages in hotfix branch

* Refactor ClientCertificateCredential construction (#8315)

* Update MSAL dependency (#8359)

* Fix InteractiveBrowserCredential caching (#8352)

* Disable depends test (#8440)

* [EventHubs] Final README update (#8430)

* [EventHubs] Final docs update (#8428)

* Readme final update

* Update links

* update async sample in readme

* Simply samples in readme and make samples can run (#8432)

* Tiny readme fix

* Update checkpointstore README

* Update checkpointstore README

* Fix a README error

* App Configuration 2019-10-01  (#8394)

* update sdk\keyvault\azure-keyvault-secrets\README.md (#8396)

* bump warden version (#8460)

* Add passing kwargs and HttpLoggingPolicy plugged (#8053)

* Live tests for managed identity, username/password, browser credentials (#7622)

* policy v2019_09_01 (#8397)

* Fix media streaming bug (#8385)

* initial commit to fix media streaming bug

- Removed media streaming related checks in synchronized_request that was causing issues with DBAs with "media" in the name
- Refactored pipeline_client.run into a _PipelineRun method for testing purposes

* bumped version

* removed references to 'Media'

* bumped version in readme

* removed unused import for pylint

* undid test config changes

* updated HISTORY with breaking changes

* modified changelog

* Work around sphinx markdown heading link bug (#8255)

* move smoke tests

update smoke-test.yml

move smoketests to common folder

* consistent smoke test requirements

* remove smoke test invocations that don't exist

* use original interface

* remove phantom smoke tests

* New CI to check autorest when changes are made in azure-core (#8509)

* ADLS Gen2 API Implementation (#8473)

* ADLS Gen2 API Implementation (#8473)

* network parameter change (#8529)

* [AutoPR] appconfiguration/resource-manager (#8528)

* New azure-mgmt-web + fixed mixin script (#8334)

* list skus test

* adding api_version in mixin instance

* regenerated new package

* regeneated tests

* history and version

* [AutoPR] cognitiveservices/data-plane/LUIS/Authoring (#7816)

* [AutoPR cognitiveservices/data-plane/LUIS/Authoring] Add v3.0 swagger (#7815)

* Generated from 7a554c128d6eba201c576885c8c2ea78f5267481

Add v3.0 swagger

* Generated from d3a4f2d1674a63bc9a4a57015f956833c7d40f7b

Update LUIS-Authoring.json

fix semantic bugs

* regenerated luis

* updated history and version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue points to a problem in the data-plane of the library. Cosmos

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants