Skip to content

feat(spanner): add OrderBy feature#10289

Merged
harshachinta merged 5 commits intogoogleapis:mainfrom
mayurkale22:orderby
Jun 7, 2024
Merged

feat(spanner): add OrderBy feature#10289
harshachinta merged 5 commits intogoogleapis:mainfrom
mayurkale22:orderby

Conversation

@mayurkale22
Copy link
Copy Markdown
Contributor

@mayurkale22 mayurkale22 commented May 30, 2024 •
edited
Loading

This feature is only applicable in Read API.

This option will allow users to specify whether they want the results to be returned in primary key order (PRIMARY_KEY) or in an out of order (NO_ORDER). The NO_ORDER is significantly faster than KEY_ORDER, because spanner uses higher parallelism for non-ordered reads.

Sample Usage

ctx := context.Background()
client, err := spanner.NewClient(ctx, db)
if err != nil {
    return err
}
defer client.Close()

iter := client.Single().ReadWithOptions(ctx, "Albums", spanner.AllKeys(),
                []string{"SingerId", "AlbumId", "AlbumTitle"}, ReadOptions.OrderBy(RpcOrderBy.NO_ORDER))
 defer iter.Stop()

@mayurkale22 mayurkale22 requested review from a team May 30, 2024 18:16
@product-auto-label product-auto-label bot added the api: spanner Issues related to the Spanner API. label May 30, 2024
harshachinta
harshachinta reviewed Jun 4, 2024
View reviewed changes
harshachinta
harshachinta reviewed Jun 4, 2024
View reviewed changes
@mayurkale22 mayurkale22 requested a review from harshachinta June 4, 2024 20:40
@mayurkale22
Copy link
Copy Markdown
Contributor Author

mayurkale22 commented Jun 6, 2024

@harshachinta, @rahul2393 Could you please review and approve this PR?

@harshachinta harshachinta enabled auto-merge (squash) June 7, 2024 04:41
@harshachinta harshachinta merged commit 07b8bd2 into googleapis:main Jun 7, 2024
@release-please release-please bot mentioned this pull request Jun 7, 2024
Merged
@mayurkale22 mayurkale22 mentioned this pull request Jun 27, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rahul2393 rahul2393 rahul2393 approved these changes

+1 more reviewer

@harshachinta harshachinta harshachinta approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

api: spanner Issues related to the Spanner API.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mayurkale22 @rahul2393 @harshachinta