Skip to content

DOCS-1197 document hashed shard keys #757

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

Closed
wants to merge 3 commits into from
Closed

DOCS-1197 document hashed shard keys #757

wants to merge 3 commits into from

Conversation

epc
Copy link
Contributor

@epc epc commented Mar 19, 2013

Signoff from dan.

including fields that hold arrays or sub-documents.

For fields that hold arrays and sub-documents, you cannot use the index to
support any query that introspects the value of an array or sub-document.
Copy link
Contributor

Choose a reason for hiding this comment

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

This isn't in the hashed index section which means it's probably a bit out of place, and also I thought re: our conversation yesterday, you couldn't use "hashed" index on multi-key fields.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You cannot create a hashed index on an array.
You can create a hashed index on apparently any other BSON type (so a: {b: {c: {d: 1, e: 2}}} is a valid document for a hashed index.

Copy link
Contributor

Choose a reason for hiding this comment

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

So "for fields that hold arrays" (which would be a multi-key index) is misleading or confusing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed references to arrays, which I thought I'd done already.

Incorporated review comments from sam
@epc
Copy link
Contributor Author

epc commented Mar 20, 2013

Pushed changes addressing comments.
note to self: don't do commit --amend once the pull req is open.

@epc
Copy link
Contributor Author

epc commented Mar 22, 2013

I think I addressed your comments, if not let me know.

@tychoish
Copy link
Contributor

I think there a few things in this list that should still be changed.

(It's ok to --amend commits in pull request, but then you have to push -f..)

@epc
Copy link
Contributor Author

epc commented Mar 22, 2013

Am closing this and doing a separate pull request.
I would prefer doing the index page refactoring as a separate ticket.

@epc epc closed this Mar 22, 2013
mongo-cr-bot pushed a commit that referenced this pull request Mar 7, 2022
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jun 16, 2025
* Update prereqs procedure

* tech review feedback

* copy and remaining tech review
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jun 23, 2025
* Update prereqs procedure

* tech review feedback

* copy and remaining tech review
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jun 26, 2025
* (DOCSP-18180) multi-cluster beta

* Initial review, content restructuring

* Build warnings removal

* Edits, ready for a copy review

* Adjust the table of contents

* A few more edits

* Address copy review round1 from John W

* Address 2nd round of copy review

* Remove exta bullet where it is not needed

* A few more cleanup type edits

* A few more nits

* Address tech review from Ciprian

* Fixed code block indentation in step 2

* Add minor edits to code examples for clarity

* Add minor edits to code examples for clarity

* Add minor edits to code examples for clarity

* Add minor edits to code examples for clarity

* Add minor edits to code examples for clarity
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jun 26, 2025
* (DOCSP-18180) multi-cluster beta

* Initial review, content restructuring

* Build warnings removal

* Edits, ready for a copy review

* Adjust the table of contents

* A few more edits

* Address copy review round1 from John W

* Address 2nd round of copy review

* Remove exta bullet where it is not needed

* A few more cleanup type edits

* A few more nits

* Address tech review from Ciprian

* Fixed code block indentation in step 2

* Add minor edits to code examples for clarity

* Add minor edits to code examples for clarity

* Add minor edits to code examples for clarity

* Add minor edits to code examples for clarity

* Add minor edits to code examples for clarity
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.

2 participants