[openstreetmap/openstreetmap-website] Do more preloading of tags and members (PR #6338)

Anton Khorev notifications at github.com
Tue Aug 26 14:07:53 UTC 2025


@AntonKhorev commented on this pull request.



> @@ -30,7 +30,7 @@ def index
 
     # Dump the details on a node given in params[:id]
     def show
-      @node = Node.find(params[:id])
+      @node = Node.includes(:element_tags).find(params[:id])
 

Does this make any difference besides running the db query earlier? Or do we want to run it earlier?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6338#pullrequestreview-3155822438
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6338/review/3155822438 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250826/1838ab79/attachment-0001.htm>


More information about the rails-dev mailing list