[GODRIVER-328] Combine the extjson and bson packages Created: 03/Apr/18 Updated: 28/Oct/23 Resolved: 23/May/18 |
|
| Status: | Closed |
| Project: | Go Driver |
| Component/s: | JSON & ExtJSON |
| Affects Version/s: | 0.0.2 |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Kristofer Brandow (Inactive) | Assignee: | Sam Kleinman (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
By merging these packages we will only add 3 functions to the BSON package. We gain the ability to stringify BSON objects to extended JSON which is easier to read than the format we currently use. We can also consolidate the types we use, eventually remove the dependency on the parser, ast, and builder packages. We will also be able to add extended JSON parsing to the decoders and add an extended JSON encoder. |
| Comments |
| Comment by Githook User [ 23/May/18 ] |
|
Author: {'username': 'tychoish', 'name': 'Sam Kleinman', 'email': 'sam@10gen.com'}Message: Change-Id: I826beae5b0af7b109138483498dbbc7463f66ea3 |
| Comment by Sam Kleinman (Inactive) [ 20/May/18 ] |
|
kris.brandow, do you think we should change the name of these three functions:
I think the last one makes a lot of sense to modify given the new package name. I'd also think that putting ExtJSON in the names of the other functions would be good, but I'm not sure if you have any patterns that you'd like to adhere to? |