[SERVER-78263] Serialize db name in the listDatabase command response with SerializationContext Created: 21/Jun/23 Updated: 29/Oct/23 Resolved: 28/Aug/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 7.1.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Sophia Tan | Assignee: | Sophia Tan |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | ntdi_must_have | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||
| Sprint: | Server Serverless 2023-06-26, Server Serverless 2023-07-10, Server Serverless 2023-07-24, Server Serverless 2023-08-07, Server Serverless 2023-08-21, Server Serverless 2023-09-04 | ||||||||||||||||||||
| Participants: | |||||||||||||||||||||
| Story Points: | 1 | ||||||||||||||||||||
| Description |
|
The listDatabase command response has db field. Currently, db name is serialized to string based on feature flag which is checked by the DatabaseNameUtil::serialize function. In this ticket, we should thread the serialization context of command request through to reply so that we can serialize the db in reply based on the "expectPrefix" and "$tenant" fields of request. For the listDatabaseForAllTenants command, we should always exclude the tenant prefix from the db of responses. |
| Comments |
| Comment by Githook User [ 27/Aug/23 ] |
|
Author: {'name': 'Sophia Tan', 'email': 'sophia_tll@hotmail.com', 'username': 'sophiatll'}Message: |
| Comment by Githook User [ 23/Aug/23 ] |
|
Author: {'name': 'Sophia Tan', 'email': 'sophia_tll@hotmail.com', 'username': 'sophiatll'}Message: Revert " This reverts commit 126f28cd8d340ada0ce7fc6afdc0e086a67621f7. |
| Comment by Githook User [ 22/Aug/23 ] |
|
Author: {'name': 'Sophia Tan', 'email': 'sophia_tll@hotmail.com', 'username': 'sophiatll'}Message: |