Skip to content

Conversation

@mobilebilly
Copy link
Contributor

This pull request implements the feature request raised in December 2020 to add support for Order Hints in Bulk Copy operations. Although the change is minor, it can result in significant performance improvements if the data being imported is sorted according to the clustered index on the destination table.

@mobilebilly
Copy link
Contributor Author

@microsoft-github-policy-service agree

@Jeffery-Wasty
Copy link
Contributor

Jeffery-Wasty commented Jul 8, 2024

Hi @mobilebilly,

We'll review this PR. If accepted. the earliest we would expect this to be merged would be the 12.9 preview sometime in August.

@Jeffery-Wasty Jeffery-Wasty linked an issue Jul 8, 2024 that may be closed by this pull request
@Jeffery-Wasty Jeffery-Wasty added this to the 12.9.0 milestone Jul 8, 2024
@Jeffery-Wasty Jeffery-Wasty changed the title Add order Hints for Bulk Copy operations (#1481) Add order Hints for Bulk Copy operations Jul 8, 2024
@mobilebilly
Copy link
Contributor Author

@Jeffery-Wasty , I am not familiar wtih the check in azure-pipelines, do I need to do anything to get them passed?

@Jeffery-Wasty
Copy link
Contributor

@mobilebilly no, this is something we need to fix on our end, we're looking into it.

@Jeffery-Wasty Jeffery-Wasty removed this from the 12.9.0 milestone Sep 21, 2024
@atul-delphix
Copy link

@Jeffery-Wasty When can we expect this feature to be released ? It's long awaited now !!!

@Jeffery-Wasty
Copy link
Contributor

Jeffery-Wasty commented Nov 6, 2024

@atul-delphix

It's tough to say. We haven't had the chance to look into this in greater detail, and would like to spend the appropriate amount of time reviewing, before releasing this into a preview. As a rough estimate, I don't think this will be included in a release this year, more likely this will come out in our first preview of 2025, but of course, this could change.

@mobilebilly
Copy link
Contributor Author

What are the next steps to proceed with this pull request?

@mobilebilly
Copy link
Contributor Author

Hi @Jeffery-Wasty @lilgreenbird @tkyc, what are the next steps to proceed with this pull request?

@David-Engel David-Engel requested review from David-Engel and removed request for Jeffery-Wasty, lilgreenbird and tkyc June 16, 2025 16:58
@David-Engel David-Engel added this to the 13.1.0 milestone Jun 16, 2025
@machavan machavan removed this from the 13.1.0 milestone Jun 26, 2025
@machavan
Copy link
Contributor

machavan commented Jul 1, 2025

Hi @mobilebilly,

We plan to merge your changes and plan to include in the next release (the one after 13.1.0 preview)

@machavan
Copy link
Contributor

Hi @mobilebilly , your PR has been merged in the main branch as : #2701

You can close this PR.

@mobilebilly
Copy link
Contributor Author

Thanks @machavan
Close this PR as the code changes have been merged in the main branch as #2701

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Under Peer Review

Development

Successfully merging this pull request may close these issues.

[FEATURE REQUEST]Add order Hints for Bulk Copy operations

5 participants