Merge from master #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixes #10634. This was mostly a regex replace of `(?<!Microsoft\.)Azure\.Search(?!\.Documents)` with `Azure.Search.Documents`. I also changed the version of the new package to `1.0.0-preview.2`.
* Update logging sample to use EventWrittenEventArgs * Use netstandard2.0-available properties
[Service Bus] ConnectionString parsing cleanup
* Regenerate SDK from latest swagger * Updating tests * Update changelog and sdk version * Update RP props
We'd like to keep the live test runs green. Marking failures `Explicit` so they can still be run locally if desired. Tracking bugs have been filed for all of them. I'm also adding Kamil as a CODEOWNER.
* move credential to Azure namespace * update doc snippets * PR feedback
* generate snowflake * update version
) * work in progress * interim commit * update api * interim split * handle multiple receipts * api update and bug fix * api tweaks * update api * Merge new CodeGen updates. * regen * api update * api tweaks * move to custom namespace * client docstrings * README * README nits * README nits * README nit * README nit * some docstrings * enable docstrings * docstrings * fix readme * another README fix * docstring updates * UX study updates * updates from ux study * api update * learned form-> page * api updates * updates for rebase * pr feedback
These changes focus on addressing several small fixes and enhancements that were being informally tracked. Included are: - Disallowing events to be added to the `EventDataBatch` when that batch is in use by an active `Send` operation. - Addition of verbiage in the `EventProcesorClient` constructor documentation to clarify that the container used by Azure Blobs storage must exist _(the processor does not assume responsibility for creation)_ - Extending `ToString` for the `EventHubsException` to include the failure reason as part of the text. - Allowing additional formats for the `Endpoint` token of the connection string when parsed; this addresses several common mistakes made when forming the string manually rather than doing copy/paste from the portal. - Trimming some extra blank lines from the `AmqpMessageConverter`.
The focus of these changes is to implement the main loop of the processor that executes in the background when the processor is running, and is responsible for coordinating with the load balancer for partition ownership changes and acting as a sentinel for the active partition processors.
* Add trigger evaluation methods and version columns * Re-record with sub ID variable
The focus of these changes is to perform a small bit of cleanup for the Event Procssor tests, removing inline try/catch blocks intended to ignore exceptions and replace them with an extension method for better formatting and readability.
* Update DotNet release to call tools from eng common directory * Change relative to absolute paths, update script parameters * Update Azure.Template to Preview 10 * Bump Azure-Template to preview 11 for testing
…10545) * Add 'Tail' to get-logs path; fix error model and update error model type * Reset the generate command to package-resources-2019-10 tag * Regenerate SDK with AutoRest version 2.0.4413 as per guidance * Add newly generated .txt file' * Remove ScriptType class since it doesn't get generated by AutoRest 2.0.4413 version Co-authored-by: Filiz Topatan <lsefitopata@microsoft.com>
* Regnerate with the latest Swagger. * Add extensions for mg-level assignments and assignment operations. * Rename subscription-level assignment test. * Record test for mg-level assignment. * Bump version to 0.20.6-preview. Co-authored-by: Shenglong Li <shenglol@microsoft.com>
* correction of description for put dataset&synchronizationsetting call * add complie log to mgmt folder * update csproj & assemblyinfo
…estrationServiceState APIs) (#10762)
The focus of these changes is to fix a set of broken links in the README documentation pertaining to identity samples.
The focus of these changes is to add a pool for transport producers used by the EventHubProducerClient which manages their lifespan using a sliding time window approach. Authored-by: Alberto De Natale <alberto.de.natale@avanade.com>
* first steps and test fix * convert unsupervised analyze result to recognized form * handle different element reference formats * pr feedback * regex updates
* Simulate Eviction API for Spot Virtual Machines * Fix NullReferenceException at CreateVM TestBase * Changing the version to a minor upgrade Co-authored-by: Sandeep Vishnu <savishnu@microsoft.com>
The focus of these changes is to add the live test coverage for those scenarios related to the `PartitionReceiver`. Behavior for the `EventHubConsumer` live tests was also tweaked in response to results from the nightly run. Test parallelism has been returned to the previous value now that the tests are more stable.
* Remove unnecessary LINQ on AmqpReceiver * Break out early * Different styling * Fix after rebase
* Adding Ashish to CODEOWNERS for storage folder * Also lining up entries as is done in other repos (whitespace only change)
Co-authored-by: Audunn Baldvinsson <audunn.baldvinsson@netapp.com>
* Fix tests broken by backend changes * Add tests into build. * Re-disable tests due to failure on Linux platform * Remove name collisions causing incorrect code generation * Add version info for package release
) Because of the transient nature of AMQP objects, there exists a race condition where an attempt to open a link my happen concurrently to the existing connection closing - either by cancellation or due to the service force-closing the connection due to timeout or other condition. Currently, this failure is detected and surfaces to callers as an `InvalidOperationException` with an unclear message. The focus of these changes is to provide resiliency in this scenario, wrapping the service exception into a form that denotes that it is safe to retry and provides messaging that is more clear for customers. In addition, because the factory for AMQP links is created as part of class-level state, it is not able to respect the cancellation token provided for individual operations. This increases the possibility of observing the race condition between link creation and closing of the connection when a cancellation token is signaled. Additional guards have been put in place to redundantly check the cancellation token before link creation to provide a better chance to respeect the token.
Fix the mixing change log entries and enable checking of them during dev builds.
… .IdentityService\AzureServiceAuth\tokenprovider.json (#11408)
…11397) * remove codes from NoWarn * tidy up more warnings * pr feedback
* Remove IEnumerable<string> ctor from SynonynMap Fixes #10725 * Used shared index to reduce resource utilization * Updated APIs and CHANGELOG
The focus of these changes is to apply a small set of improvements to the
various README instances within the Event Hubs source. The specific changes
made were:
- Improve the wording and reference materials for the Azure Subscription
prerequisite.
- Reword the C# 8 prerequisite to help clarify the trade-offs for choosing
a language version, and ensure that it is clear the library can be used
with language versions before 8.
- Remove the culture slugs from the reference links, allowing those who
follow to be redirected to their currently active culture.
By default, the `.Parse` and `.TryParse` methods, as well as `.ToString` use the current culture. In some of our uses, we don't want to the behavior to differ based on the culture of the local machine.
* trainig argument and live tests * PR feedbacl * fixes * Wes' work for test infra * pr feedback
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.