[EventHubs] Eventhubs Sync Changes #8336
Merged
Conversation
This file contains hidden or 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
|
Can one of the admins verify this patch? |
annatisch
reviewed
Nov 1, 2019
sdk/eventhub/azure-eventhubs/azure/eventhub/_eventprocessor/partition_context.py
Show resolved
Hide resolved
sdk/eventhub/azure-eventhubs/azure/eventhub/_eventprocessor/event_processor.py
Outdated
Show resolved
Hide resolved
sdk/eventhub/azure-eventhubs/azure/eventhub/_eventprocessor/event_processor.py
Outdated
Show resolved
Hide resolved
sdk/eventhub/azure-eventhubs/azure/eventhub/_eventprocessor/event_processor.py
Outdated
Show resolved
Hide resolved
sdk/eventhub/azure-eventhubs/azure/eventhub/_eventprocessor/event_processor.py
Outdated
Show resolved
Hide resolved
sdk/eventhub/azure-eventhubs/azure/eventhub/_eventprocessor/event_processor.py
Outdated
Show resolved
Hide resolved
lmazuel
reviewed
Nov 4, 2019
sdk/eventhub/azure-eventhubs/tests/livetest/synctests/test_consumer_client.py
Show resolved
Hide resolved
lmazuel
reviewed
Nov 4, 2019
lmazuel
reviewed
Nov 4, 2019
lmazuel
reviewed
Nov 4, 2019
lmazuel
reviewed
Nov 4, 2019
lmazuel
changed the base branch from
eventhubs_preview5
to
feature/eventhubs_preview5
lmazuel
reviewed
Nov 4, 2019
lmazuel
reviewed
Nov 4, 2019
yunhaoling
pushed a commit
that referenced
this pull request
Nov 5, 2019
* [EventHubs] Eventhubs Sync Changes (#8336) * Sync EventHub API for review * 2.7 compatible * new api tests * Update sync version implemention and tests * Making CI happy * remove async file and add README CHANGELOG * Update common files in eventprocessor * Adding checkpointstoreblob * Update unittest and share_requirement * Update shared_requirements.txt * Update setup.py for CI to pass * Update storage dependency * Update according to comment * [EventHubs] Eventhubs aio changes (#8331) * Eventhubs AIO changes * Add new files * Add new files * Add test code for review * Changes for code review * Make it testable * Update conftest for backward compatibility * add init to _eventprocessor * Remove obsolete async test example * Remove sample partition manager * update blobstorageaio * Put close reason and ownership error to common * Add blobstorage stuffs * fix pylint * Fix shared_requirement * Change blob storage dependency to >=12.0.0 * small doc change * Doc for eventhub * More doc improvement * Fix doc folder detection * Doc improvement * [EventHubs] Updating samples, small update in readme (#8383) * Updating samples, small update in readme * Small update on blob storage sample * Update readme to fix analyze check * remove timeout and retry in sample * fix pylint and update rst * remove timeout and retry in sample * Remove CloseReason from aio init * Update docstring * Hide static keys of EventData * Update from connection string and test.yml (#8392) * update env name in yml * update from connection string * small fix * Update docstring * Fix a test file problem after merge async/sync * Docstring, sample, readme update (#8395) * update env name in yml * update from connection string * small fix * Update manifest files, samples, code snippet * Fix in docstring * update readme * Small fix * remove unused import * sync and async sample balance load * Changelog, sample, test updates (#8411) * doc feedback update * update on changelog and samples * small fix in samples * Revert "Fix doc folder detection" This reverts commit b3c601f.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
Major changes:
The public classes are: