Skip to content

DOCSP-42961: Removed Nested Components #581

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.

Sign up for GitHub

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

Merged
merged 15 commits into from
Oct 23, 2024
Merged

DOCSP-42961: Removed Nested Components #581

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
577890a
DOCSP-42961: Removed Nested Components
caitlindavey Oct 18, 2024
a54e603
Fixing indentation errors
caitlindavey Oct 18, 2024
8fb5894
Fixing indentation errors again
caitlindavey Oct 18, 2024
6b40e89
Fixing indentation errors again
caitlindavey Oct 18, 2024
07a82f5
Fixing indentation errors again
caitlindavey Oct 18, 2024
7103e2f
Fixing indentation errors again+
caitlindavey Oct 18, 2024
19bdf64
Fixing indentation errors again+
caitlindavey Oct 18, 2024
38db069
Fixing code block indentation
caitlindavey Oct 18, 2024
4f85b13
tweaks based on internal review feedback
caitlindavey Oct 22, 2024
75f8585
tweaks based on internal review feedback - indentation issues
caitlindavey Oct 22, 2024
4262df6
tweaks based on internal review feedback - indentation issues
caitlindavey Oct 22, 2024
697f184
tweaks based on internal review feedback - indentation issues
caitlindavey Oct 22, 2024
3ebef66
tweaks based on internal review feedback - indentation issues
caitlindavey Oct 22, 2024
646b22e
tweaks to bullet issue
caitlindavey Oct 22, 2024
9a899e7
tweaks to bullet issue
caitlindavey Oct 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 16 additions & 19 deletions source/fundamentals/connection/mongoclientsettings.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -259,26 +259,23 @@ settings to modify the driver's behavior:

* - ``serverSelector()``
- Adds a server selector to apply before server selection.

* - ``srvHost()``
- Sets the host name to use to look up an SRV DNS record to find the
MongoDB hosts.

.. note::

When setting ``srvHost``, the driver does not process any
associated TXT records associated with the host.

If you want to enable the processing of TXT records, you must
specify the SRV host in the connection string using the
``applyConnectionString()`` method.

.. code-block:: java
:emphasize-lines: 3

MongoClient mongoClient = MongoClients.create(
MongoClientSettings.builder()
.applyConnectionString(new ConnectionString("mongodb+srv://host1.acme.com")))
- | Sets the host name to use to look up an SRV DNS record to find the
MongoDB hosts.
|
| If you want to enable the processing of TXT records associated with the host,
specify the SRV host in the connection string
using the ``applyConnectionString()`` method.
|
| For example:

.. code-block:: java
:emphasize-lines: 3

MongoClient mongoClient =
MongoClients.create(MongoClientSettings.builder()
.applyConnectionString(new ConnectionString("mongodb+srv://host1.acme.com")))

* - ``srvMaxHosts()``
- | Sets the maximum number of hosts the driver can connect to when using
Expand Down
40 changes: 16 additions & 24 deletions source/fundamentals/data-formats/pojo-customization.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -285,11 +285,7 @@ package:

* - ``BsonRepresentation``
- Specifies the BSON type used to store the value when different from the
POJO property.

.. seealso::

:ref:`bsonrepresentation-annotation-code-example`
POJO property. See an example of :ref:`bsonrepresentation-annotation-code-example` on this page.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
POJO property. See an example of :ref:`bsonrepresentation-annotation-code-example` on this page.
POJO property. See an example of a :ref:`bsonrepresentation-annotation-code-example` on this page.

S: Flows a bit better grammatically with the "a"


* - ``BsonId``
- Marks a property to serialize as the _id property.
Expand All @@ -299,28 +295,24 @@ package:
and/or deserialize a property.

* - ``BsonProperty``
- Specifies a custom document field name when converting the POJO
field to BSON. You can include a discriminator to serialize POJOs
nested within the field.

.. important::
- | Specifies a custom document field name when converting the POJO
field to BSON. You can include a discriminator to serialize POJOs
nested within the field.
|
| When applying ``@BsonProperty`` to a private field,
**you must also add getter and setter methods for that field to serialize and
customize the field name**.
Comment on lines +302 to +304
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
| When applying ``@BsonProperty`` to a private field,
**you must also add getter and setter methods for that field to serialize and
customize the field name**.
| When applying ``@BsonProperty`` to a private field,
**you must also add getter and setter methods for that field to serialize and
customize the field name**.

S: The bolding looks a little too bold for this small section. The Style Guide suggested using the gold Important directive to replace the Important note directive: https://www.mongodb.com/docs/meta/style-guide/nested-components/#important-or-warning.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These are being removed as part of this nested component work where we want annotations taken out of tables.

Copy link
Collaborator

@lindseymoore lindseymoore Oct 22, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Is using the gold Important directive rather than the bolding not necessary here? Meant this gold directive that the style guide recommends: https://www.mongodb.com/docs/meta/style-guide/nested-components/#important-or-warning:~:text=IMPORTANT%3A%20Data%20transfer%20as%20part%20of%20the%20backup%20and%20restore%20process%20is%20charged%20separately.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ok, LGTM!


When applying ``@BsonProperty`` to a private field, you must also add
getter and setter methods for that field to serialize and customize
the field name.

* - ``BsonExtraElements``
- Specifies the POJO field on which to deserialize all elements that are
not mapped to a field. The POJO field must be one of the following
types:

- `Document <{+api+}/apidocs/bson/org/bson/Document.html>`__
- `BsonDocument <{+api+}/apidocs/bson/org/bson/BsonDocument.html>`__
- ``Map<String, Object>``

.. seealso::

:ref:`BsonExtraElements Annotation Example <bsonextraelements-annotation-code-example>`
- | Specifies the POJO field on which to deserialize all elements that are
not mapped to a field. The POJO field must be one of the following
types:
| - `Document <{+api+}/apidocs/bson/org/bson/Document.html>`__
| - `BsonDocument <{+api+}/apidocs/bson/org/bson/BsonDocument.html>`__
| - ``Map<String, Object>``
|
| See an example of a :ref:`BsonExtraElements Annotation Example <bsonextraelements-annotation-code-example>`.

The following code snippet shows a sample POJO called ``Product`` that uses
several of the preceding annotations.
Expand Down
Loading