Skip to content

Conversation

sharadregoti
Copy link
Contributor

@sharadregoti sharadregoti commented Jun 10, 2025

User description

Preview Link

Checklist

  • Added a preview link
  • Reviewed AI PR Agent suggestions
  • For Tyk Members - Added Jira DX PR ticket to the subject
  • For Tyk Members - Added the appropriate release labels (for fixes add the latest release)

New Contributors


PR Type

Documentation


Description

  • Removed outdated note on record size and storage growth

  • Clarified current Graph MongoDB Pump limitations

  • Improved documentation accuracy for Tyk Pump users


Changes walkthrough 📝

Relevant files
Documentation
tyk-pump.md
Remove outdated storage size warning from documentation   

tyk-docs/content/api-management/tyk-pump.md

  • Deleted note about record size and MongoDB storage growth
  • Retained and clarified other current limitations of Graph MongoDB Pump
  • +0/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • Copy link
    Contributor

    ⚠️ Deploy preview for PR #6510 did not become live after 3 attempts.
    Please check Netlify or try manually: Preview URL

    @sharadregoti
    Copy link
    Contributor Author

    This PR fixes #5419

    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Documentation Accuracy

    Ensure that the removal of the note regarding record size and storage growth does not omit important information for users who may need to be aware of potential storage implications when using the Graph MongoDB Pump.

    The Graph MongoDB Pump is being improved upon regularly and as such there are a few things to note about the Graph MongoDB Pump current behavior:
    
    * Subgraph requests are not recorded - Requests to tyk-controlled subgraphs from supergraphs in federation setting are currently not recorded by the Graph MongoDB Pump, just the supergraph requests are handled by the Graph MongoDB Pump.
    * UDG requests are recorded but subsequent requests to data sources  are currently ignored.
    * Currently, Graph MongoDB Pump data can not be used in Tyk Dashboard yet, the data is only stored for recording purposes at the moment and can be exported to external tools for further analysis.

    Copy link
    Contributor

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Copy link

    netlify bot commented Jun 10, 2025

    PS. Add to the end of url /docs/nightly

    Name Link
    🔨 Latest commit 32de48d
    🔍 Latest deploy log https://app.netlify.com/projects/tyk-docs/deploys/6847d47ea4bfb60008b7dff2
    😎 Deploy Preview https://deploy-preview-6510--tyk-docs.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 project configuration.

    Copy link

    netlify bot commented Jun 10, 2025

    PS. Add to the end of url /docs/nightly

    Name Link
    🔨 Latest commit 3ec77dd
    🔍 Latest deploy log https://app.netlify.com/projects/tyk-docs/deploys/68540a666a0428000844bfa2
    😎 Deploy Preview https://deploy-preview-6510--tyk-docs.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 project configuration.

    @sharadregoti sharadregoti marked this pull request as draft June 19, 2025 14:50
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants