[SERVER-23611] Query planner should set collation from collection default if query lacks user-specified collation Created: 07/Apr/16  Updated: 14/Jul/16  Resolved: 13/Jun/16

Status: Closed
Project: Core Server
Component/s: Querying
Affects Version/s: None
Fix Version/s: 3.3.9

Type: Task Priority: Major - P3
Reporter: J Rassi Assignee: J Rassi
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
depends on SERVER-23610 CanonicalQuery should own a CollatorI... Closed
depends on SERVER-23849 Add collection default collation to t... Closed
is depended on by SERVER-23924 Make _id index inherit the collection... Closed
Related
is related to SERVER-24534 Commands that accept user predicates ... Closed
Backwards Compatibility: Fully Compatible
Sprint: Query 13 (04/22/16), Query 14 (05/13/16), Query 15 (06/03/16), Query 16 (06/24/16)
Participants:

 Comments   
Comment by Githook User [ 13/Jun/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 Collection-default jstests for collation-aware ops
Branch: master
https://github.com/mongodb/mongo/commit/2bf2c9d2d5de6ff8136bfe9ca30346300deadb13

Comment by Githook User [ 13/Jun/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 Agg use collection default collation, if unspecified
Branch: master
https://github.com/mongodb/mongo/commit/6b6cd6727d262d5db5e4f226e4da0d2bc410a4d8

Comment by Githook User [ 13/Jun/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 Distinct use collection default collation, if unspecified
Branch: master
https://github.com/mongodb/mongo/commit/848949b47b5af8bb0302ee1b8dbbbcac47969d42

Comment by Githook User [ 13/Jun/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 Queries use collection default collation, if unspecified
Branch: master
https://github.com/mongodb/mongo/commit/a63ddd2455e456c602e2ba9a4f4da0a424068f94

Comment by Githook User [ 10/Jun/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 Add InMatchExpression::setCollator() method

Also removes collator parameter from InMatchExpression constructor.
Branch: master
https://github.com/mongodb/mongo/commit/d5e278328c445b15f403312a41e82bec342e74fd

Comment by Githook User [ 06/Jun/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 Improve signature of prepareExecution()

Instead of prepareExecution() taking two out-parameters and mutating
one of its in-parameters, it now explicitly takes ownership of the
CanonicalQuery passed in and returns a StatusWith<> that wraps the
three return values. Forcing callers to std::move() their
CanonicalQuery and extract it back out should help make it more
obvious that it may be mutated.
Branch: master
https://github.com/mongodb/mongo/commit/82425c23f333636af821af38fc44979038b14d50

Comment by Githook User [ 06/Jun/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 Add collation tests for non-existent collections
Branch: master
https://github.com/mongodb/mongo/commit/3a22c6045500cd7c7d1f76618f5a462bfc17d386

Comment by Githook User [ 06/Jun/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 Add ComparisonMatchExpression::setCollator() method

Also removes collator parameter from ComparisonMatchExpression
constructor.
Branch: master
https://github.com/mongodb/mongo/commit/c78f95141142e4d13c820a64aa26fe28d07ebb0c

Comment by Githook User [ 06/Jun/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 MatchExpressionParser::_parse() should take a collator

Removes member MatchExpressionParser::_collator.
Branch: master
https://github.com/mongodb/mongo/commit/38ab06d09b2295d404c7e8d012a8491ed0a0ec1f

Comment by Githook User [ 27/May/16 ]

Author:

{u'username': u'jrassi', u'name': u'J. Rassi', u'email': u'rassi@10gen.com'}

Message: SERVER-23611 Refactor InMatchExpression

Includes deletion of the ArrayFilterEntries class.

This is groundwork for implementing InMatchExpression::setCollator().
Branch: master
https://github.com/mongodb/mongo/commit/f93ad728e94b6813c242faac43d4b57a07a3c319

Generated at Thu Feb 08 04:03:54 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.