Skip to content

Reduce MTC feed updater query - #663

Merged
binh-dam-ibigroup merged 10 commits into
devfrom
reduce-mtc-feed-updater-query
Aug 12, 2026
Merged

Reduce MTC feed updater query#663
binh-dam-ibigroup merged 10 commits into
devfrom
reduce-mtc-feed-updater-query

Conversation

@binh-dam-ibigroup

@binh-dam-ibigroup binh-dam-ibigroup commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Checklist

  • Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
  • Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
  • The description lists all applicable issues this PR seeks to resolve
  • [na] The description lists any configuration setting(s) that differ from the default settings
  • All tests and CI builds passing

Description

Query only select fields to reduce the memory used in Mongo while querying for feed version updates.

@binh-dam-ibigroup
binh-dam-ibigroup marked this pull request as ready for review August 10, 2026 18:47

@br648 br648 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All good, just a couple of minor items to consider.

Comment thread src/main/java/com/conveyal/datatools/manager/jobs/FeedUpdater.java
Comment thread src/main/java/com/conveyal/datatools/manager/jobs/FeedUpdater.java Outdated
@br648 br648 assigned binh-dam-ibigroup and unassigned br648 Aug 11, 2026
@binh-dam-ibigroup
binh-dam-ibigroup merged commit 7ba13db into dev Aug 12, 2026
6 of 7 checks passed
@binh-dam-ibigroup
binh-dam-ibigroup deleted the reduce-mtc-feed-updater-query branch August 12, 2026 20:50
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.

3 participants