Skip to content

Conversation

@YijunXieMS
Copy link
Contributor

Merge to master and get ready to release
detailed description to be added...

YijunXieMS and others added 3 commits July 26, 2019 16:30
* Shared connection (sync) draft

* Shared connection (sync) draft 2

* Shared connection (sync) test update

* Shared connection

* Fix an issue

* add retry exponential delay and timeout to exception handling

* put module method before class def

* fixed Client.get_properties error

* Improve send timeout (#6481)

* Add timeout information to the link prop during link creation process (#6485)

* Update optional parameters in public api into kwargs and update comments (#6510)

* Update optional parameters in public api into kwargs and update some comments

* Update more optional parameters into kwargs paramter
* create_batch feature implementation (#6256)

* Init create batch event

* create_batch implementation

* Revert _set_partition_key method and update comment

* Refacor EventDataBatch class

* Revert logic when setting partition_key of event_data

* Misc fixes from code review

* Rename max_message_size to max_size
Other small fixes

* Warn if event_data is None when call try_add

* Create batch event update (#6509)

* Update according to the review

* Update comment

* Revert some kwargs backto optional parameters as it may cause breaking changes

* Small fixes (#6520)

* Change back to normal number writings as not supported by python under 3.6

* small fix

* Add missing return (#6522)

* Fix livetest (#6523)
* Shared connection (sync) draft

* Shared connection (sync) draft 2

* Shared connection (sync) test update

* Shared connection

* Fix an issue

* add retry exponential delay and timeout to exception handling

* put module method before class def

* fixed Client.get_properties error

* new eph (draft)

* new eph (draft2)

* remove in memory partition manager

* EventProcessor draft 3

* small format change

* Fix logging

* Add EventProcessor example

* use decorator to implement retry logic and update some tests (#6544)

* Update livetest (#6547)

* Remove legacy code and update livetest (#6549)

* Update livetest

* Remove legacy code and update livetest

* make sync longrunning multi-threaded

* small changes on async long running test

* reset retry_count for iterator

* Don't return early when open a ReceiveClient or SendClient

* type annotation change

* Update kwargs and remove unused import

* Misc changes from EventProcessor PR review

* raise asyncio.CancelledError out instead of supressing it.

* Update livetest and small fixed (#6594)

* Add missing close in livetest

* Update livetest to wait longer

* Close handler each time before retry

* Fix feedback from PR (1)

* Revert "Merge branch 'eventhubs_dev' into eventhubs_eph"

This reverts commit 19a5539, reversing
changes made to 9d18dd9.

* Fix feedback from PR (2)

* Update code according to the review (#6623)

* Wait longer for reconnect op

* Raise authentication error when open timeout

* Optimize retry decorator

* Update code according to review

* Small fix

* Fix feedback from PR (3)

* small bug fixing

* Remove old EPH

*  Update decorator implementation (#6642)

* Update decorator implementation

* Remove old EPH pytest

* Revert "Revert "Merge branch 'eventhubs_dev' into eventhubs_eph""

This reverts commit d688090.

* Update sample codes and docstring (#6643)

* Check tablename to prevent sql injection

* PR review update

* Removed old EPH stuffs.
Added new EPH stuffs.

* Small fix (#6650)

* Draft for changelog

* Improve syntax for kwargs
@YijunXieMS YijunXieMS added Event Hubs Client This issue points to a problem in the data-plane of the library. labels Aug 5, 2019
@adxsdk6
Copy link

adxsdk6 commented Aug 5, 2019

Can one of the admins verify this patch?

@YijunXieMS YijunXieMS requested a review from xiangyan99 August 5, 2019 21:30
Copy link
Member

@xiangyan99 xiangyan99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please take a look

yijxie and others added 4 commits August 5, 2019 16:39
* Draft for updating documentations

* Small improvement

* Updating docstrings, docs and sample
@yunhaoling yunhaoling requested a review from ramya-rao-a August 6, 2019 00:43
@yunhaoling
Copy link
Contributor

@ramya-rao-a

Hello Ramya, please review our readme and history file.

Copy link
Contributor

@yunhaoling yunhaoling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@YijunXieMS YijunXieMS merged commit 8de236a into master Aug 6, 2019
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. Event Hubs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants