Skip to content

DOCSP-46984: Upcoming features #19

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

Merged
merged 15 commits into from
Feb 3, 2025

Conversation

norareidy
Copy link
Collaborator

@norareidy norareidy commented Feb 3, 2025

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-46984

Staging Links

  • index
  • limitations-upcoming
  • Self-Review Checklist

    • Is this free of any warnings or errors in the RST?
    • Did you run a spell-check?
    • Did you run a grammar-check?
    • Are all the links working?
    • Are the facets and meta keywords accurate?

    Copy link

    netlify bot commented Feb 3, 2025

    Deploy Preview for docs-django ready!

    Name Link
    🔨 Latest commit d307fa0
    🔍 Latest deploy log https://app.netlify.com/sites/docs-django/deploys/67a107c5fae1f100071fbe07
    😎 Deploy Preview https://deploy-preview-19--docs-django.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify site configuration.

    @R-shubham R-shubham requested review from Jibola and R-shubham February 3, 2025 16:44
    @rustagir rustagir self-requested a review February 3, 2025 16:55
    - Performing Vector Search, Atlas Search, and geospatial queries
    - Encrypting data by using Queryable Encryption and Client-side Field Level Encryption
    - Running database transactions
    - Using specialized aggregation operators
    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    We can remove this for now.

    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    "Using specialized aggregation operators"

    - Using GridFS to store large files
    - Opening change streams to monitor your data
    - Storing Django's cached data in the database
    - Specifying schema validation
    - Using specialized aggregation operators
    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    "Using specialized aggregation operators" - Let's remove this for now.

    Copy link
    Collaborator

    @Jibola Jibola left a comment

    Choose a reason for hiding this comment

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

    Few comments! Everything else looks great

    the upcoming GA release:

    - Creating Vector Search, Atlas Search, and geospatial indexes
    - Performing Vector Search, Atlas Search, and geospatial queries
    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    This could be misleading. We should either remove it or change the language to say "improved" or "through the Django API"

    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    Performing Vector Search, Atlas Search, and geospatial queries directly through the Django API

    We plan to support the following model features in
    future post-GA releases:

    - Interacting with ``JSONField`` values without the limitations on ``Q`` object queries and
    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    Let's remove this. We have not promised this.

    - ``dumpdata``
    - ``loaddata``

    We plan to support the following commands in
    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    We can remove these two. We may not commit to adding these.

    Copy link
    Collaborator

    @R-shubham R-shubham left a comment

    Choose a reason for hiding this comment

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

    LGTM, once the suggested changes from me and Jib are incorporated.

    ~~~~~~~~~~~~~~

    We plan to support the following model features in
    upcoming releases:
    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    Can we remove all upcoming with "GA"/ Post-GA release? Upcoming may sound like the next release. We will have many smaller releases leading up to the final GA release.

    Copy link
    Collaborator

    @R-shubham R-shubham left a comment

    Choose a reason for hiding this comment

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

    left a comment around wording for upcoming release

    Copy link

    @rustagir rustagir left a comment

    Choose a reason for hiding this comment

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

    a few things, and I can also wait to re-review until you resolve the tech feedback so there arent any redundant issues

    In this section, you can learn about the following
    types of {+framework+} and MongoDB features that
    General Availability (GA) and post-GA {+django-odm+} releases
    will likely introduce:
    Copy link

    Choose a reason for hiding this comment

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

    Q: is there a better way to phrase this to be more authoritative? What's the limitation to just saying these features will be supported?

    Copy link
    Collaborator Author

    Choose a reason for hiding this comment

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

    None of these features are guaranteed, so I was asked to use non-authoritative language

    Comment on lines 269 to 270
    - Creating Vector Search, Atlas Search, and geospatial indexes
    - Performing Vector Search, Atlas Search, and geospatial queries
    Copy link

    Choose a reason for hiding this comment

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

    Suggested change
    - Creating Vector Search, Atlas Search, and geospatial indexes
    - Performing Vector Search, Atlas Search, and geospatial queries
    - Programmatic management of Vector Search, Atlas Search, and geospatial indexes
    - Vector Search, Atlas Search, and geospatial queries


    - Creating Vector Search, Atlas Search, and geospatial indexes
    - Performing Vector Search, Atlas Search, and geospatial queries
    - Encrypting data by using Queryable Encryption and Client-side Field Level Encryption
    Copy link

    Choose a reason for hiding this comment

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

    Suggested change
    - Encrypting data by using Queryable Encryption and Client-side Field Level Encryption
    - Queryable Encryption and Client-side Field Level Encryption

    Comment on lines 272 to 273
    - Running database transactions
    - Storing Django's cached data in the database
    Copy link

    Choose a reason for hiding this comment

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

    Suggested change
    - Running database transactions
    - Storing Django's cached data in the database
    - Database transactions
    - Storage of {+framework+}'s cached data in the database

    We plan to support the following MongoDB features in
    future post-GA releases:

    - Using GridFS to store large files
    Copy link

    Choose a reason for hiding this comment

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

    S: modify list to be nouns as you call them features instead of actions

    @norareidy norareidy requested review from rustagir and Jibola February 3, 2025 18:08
    Copy link

    @rustagir rustagir left a comment

    Choose a reason for hiding this comment

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

    looks good!

    Comment on lines 317 to 320
    We plan to support the following model features in
    future post-GA releases:

    - Custom ``Field`` class for the ``BSONRegExp`` data type
    Copy link

    Choose a reason for hiding this comment

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

    s: consider converting list of 1 into a sentence

    @norareidy norareidy merged commit 89d3145 into mongodb:master Feb 3, 2025
    5 checks passed
    @norareidy norareidy deleted the DOCSP-46984-upcoming-features branch February 3, 2025 18:20
    norareidy added a commit that referenced this pull request Feb 3, 2025
    * DOCSP-46984: Upcoming features
    
    * landing page
    
    * edits
    
    * shorten
    
    * edit intro
    
    * edits
    
    * fix links
    
    * typo
    
    * cache
    
    * ga and post ga
    
    * wording
    
    * feedback
    
    * wording
    
    * verb tense
    
    * RR feedback 2
    
    (cherry picked from commit 89d3145)
    norareidy added a commit that referenced this pull request Feb 3, 2025
    * DOCSP-46984: Upcoming features
    
    * landing page
    
    * edits
    
    * shorten
    
    * edit intro
    
    * edits
    
    * fix links
    
    * typo
    
    * cache
    
    * ga and post ga
    
    * wording
    
    * feedback
    
    * wording
    
    * verb tense
    
    * RR feedback 2
    
    (cherry picked from commit 89d3145)
    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.

    4 participants