Skip to content
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

Vendor the azblob 0.6.0 dependency so customers can upgrade to later blobs packages #262

Merged
merged 12 commits into from
Oct 21, 2022

Conversation

richardpark-msft
Copy link
Member

@richardpark-msft richardpark-msft commented Jul 26, 2022

azblob@v0.6.0 is a beta dependency we use for storage checkpointing. More recent releases won't compile with our package, which means that customers that want to move forward can't.

Vendoring this dependency allows us to avoid doing that upgrade until the blob package has finalized, or until we migrate over to the latest package released in azure-sdk-for-go.

Fixes #261

@jhendrixMSFT jhendrixMSFT force-pushed the ehold-vendor-storage branch 4 times, most recently from 096ee84 to fb63fff Compare July 26, 2022 22:25
@richardpark-msft richardpark-msft changed the title Ehold vendor storage Vendor the azblob 0.6.0 dependency so customers can upgrade to later blobs packages Jul 26, 2022
@richardpark-msft richardpark-msft marked this pull request as ready for review July 26, 2022 22:39
@richardpark-msft
Copy link
Member Author

/azp run Azure.azure-event-hubs-go

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@jhendrixMSFT jhendrixMSFT force-pushed the ehold-vendor-storage branch from fb63fff to 0b70bda Compare July 26, 2022 22:51
@jhendrixMSFT jhendrixMSFT self-requested a review July 26, 2022 22:51
@jhendrixMSFT
Copy link
Member

/azp run Azure.azure-event-hubs-go

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@jhendrixMSFT
Copy link
Member

/azp run Azure.azure-event-hubs-go

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

…eTypes are strings they don't "match" when it comes to determining if they're the same type. Aliasing should work just fine here.
@aarshshah-8
Copy link

aarshshah-8 commented Oct 20, 2022

When will this be merged in? I have an issue using an updated snowflakedb (1.6.13) which needs azure-blobs 0.14.0 but azure-events-hub still requires azure-blobs 0.6.0, any recommended work arounds for such issues which leads the issues mentioned in
#261

@jhendrixMSFT jhendrixMSFT merged commit adc9788 into Azure:master Oct 21, 2022
@jhendrixMSFT
Copy link
Member

@aarshshah-8 sorry for the delay, this has been tagged and released

@aarshshah-8
Copy link

@aarshshah-8 sorry for the delay, this has been tagged and released

No worries! Thank you for taking care of it so quick. Tested it out and it works great

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using a newer version of azblob causes a compile error
3 participants