hummeltech left a comment (openstreetmap/mod_tile#501)

Thank you very much for the comprehensive write-up for this issue.


I have implemented the changes you requested:
    Before:
```shell
> build/src/render_list --all --config build/tests/conf/file/renderd.conf 
> --force --map default --num-threads 4 \
        --min-zoom 0 --max-zoom 8 \
        --min-lon -125.0 --max-lon -66.9 --min-lat 24.4 --max-lat 49.4
** INFO: 07:29:24.402: Started render_list with the following options:
** INFO: 07:29:24.403:  --config      = 'build/tests/conf/file/renderd.conf' 
(user-specified)
** INFO: 07:29:24.403:  --map         = 'default' (user-specified)
** INFO: 07:29:24.403:  --max-load    = '16' (default)
** INFO: 07:29:24.403:  --max-zoom    = '8' (user-specified/from config)
** INFO: 07:29:24.403:  --min-zoom    = '0' (user-specified/from config)
** INFO: 07:29:24.403:  --num-threads = '4' (user-specified/from config)
** INFO: 07:29:24.403:  --socket      = 'build/tests/run/file/renderd0.sock' 
(user-specified/from config)
** INFO: 07:29:24.403:  --tile-dir    = 'build/tests/tiles/file' 
(user-specified/from config)
** Message: 07:29:24.403: Starting 4 rendering threads
** Message: 07:29:24.403: Rendering all tiles from zoom 0 to zoom 8
** Message: 07:29:24.403: Rendering all tiles for zoom 0 from (0, 0) to (0, 0)
** Message: 07:29:24.403: Rendering all tiles for zoom 1 from (0, 0) to (0, 0)
** Message: 07:29:24.403: Rendering all tiles for zoom 2 from (0, 1) to (0, 1)
** Message: 07:29:24.403: Rendering all tiles for zoom 3 from (1, 2) to (1, 3)
** Message: 07:29:24.403: Rendering all tiles for zoom 4 from (2, 5) to (4, 6)
** Message: 07:29:24.403: Rendering all tiles for zoom 5 from (4, 10) to (9, 13)
** Message: 07:29:24.403: Rendering all tiles for zoom 6 from (9, 21) to (19, 
27)
** Message: 07:29:24.403: Rendering all tiles for zoom 7 from (19, 43) to (39, 
55)
** Message: 07:29:24.475: Rendering all tiles for zoom 8 from (39, 87) to (79, 
110)
** Message: 07:29:29.454: Waiting for rendering threads to finish
** Message: 07:29:31.455: Total for all tiles rendered
** Message: 07:29:31.455: Metatiles rendered:
** Message: 07:29:31.455:       32 in 7.05 seconds (4.54/s)
** Message: 07:29:31.455: Total tiles rendered:
** Message: 07:29:31.455:       2048 in 7.05 seconds (290.40/s)
** Message: 07:29:31.455: Total tiles handled:
** Message: 07:29:31.455:       32 in 7.05 seconds (4.54/s)
```
    
    After applying the fix for "cause 1":
```shell
> build/src/render_list --all --config build/tests/conf/file/renderd.conf 
> --force --map default --num-threads 4 \
        --min-zoom 0 --max-zoom 8 \
        --min-lon -125.0 --max-lon -66.9 --min-lat 24.4 --max-lat 49.4
** INFO: 20:51:33.251: Started render_list with the following options:
** INFO: 20:51:33.251:  --config      = 'build/tests/conf/file/renderd.conf' 
(user-specified)
** INFO: 20:51:33.251:  --map         = 'default' (user-specified)
** INFO: 20:51:33.251:  --max-load    = '16' (default)
** INFO: 20:51:33.251:  --max-zoom    = '8' (user-specified/from config)
** INFO: 20:51:33.251:  --min-zoom    = '0' (user-specified/from config)
** INFO: 20:51:33.251:  --num-threads = '4' (user-specified/from config)
** INFO: 20:51:33.251:  --socket      = 'build/tests/run/file/renderd0.sock' 
(user-specified/from config)
** INFO: 20:51:33.251:  --tile-dir    = 'build/tests/tiles/file' 
(user-specified/from config)
** Message: 20:51:33.251: Starting 4 rendering threads
** Message: 20:51:33.251: Rendering all tiles from zoom 0 to zoom 8
** Message: 20:51:33.251: Rendering all tiles for zoom 0 from (0, 0) to (0, 0)
** Message: 20:51:33.251: Rendering all tiles for zoom 1 from (0, 0) to (0, 0)
** Message: 20:51:33.251: Rendering all tiles for zoom 2 from (0, 1) to (0, 1)
** Message: 20:51:33.251: Rendering all tiles for zoom 3 from (1, 2) to (1, 3)
** Message: 20:51:33.251: Rendering all tiles for zoom 4 from (2, 5) to (4, 6)
** Message: 20:51:33.251: Rendering all tiles for zoom 5 from (4, 10) to (9, 13)
** Message: 20:51:33.251: Rendering all tiles for zoom 6 from (9, 21) to (19, 
27)
** Message: 20:51:33.281: Rendering all tiles for zoom 7 from (19, 43) to (39, 
55)
** Message: 20:51:34.309: Rendering all tiles for zoom 8 from (39, 87) to (79, 
110)
** Message: 20:51:40.311: Waiting for rendering threads to finish
** Message: 20:51:42.311: Total for all tiles rendered
** Message: 20:51:42.311: Metatiles rendered:
** Message: 20:51:42.311:       41 in 9.06 seconds (4.53/s)
** Message: 20:51:42.311: Total tiles rendered:
** Message: 20:51:42.311:       2624 in 9.06 seconds (289.62/s)
** Message: 20:51:42.311: Total tiles handled:
** Message: 20:51:42.311:       41 in 9.06 seconds (4.53/s)
```


    After applying the fix for "cause 2":
    
```shell
> build/src/render_list --all --config build/tests/conf/file/renderd.conf 
> --force --map default --num-threads 4 \
        --min-zoom 0 --max-zoom 8 \
        --min-lon -125.0 --max-lon -66.9 --min-lat 24.4 --max-lat 49.4
** INFO: 20:49:27.495: Started render_list with the following options:
** INFO: 20:49:27.495:  --config      = 'build/tests/conf/file/renderd.conf' 
(user-specified)
** INFO: 20:49:27.495:  --map         = 'default' (user-specified)
** INFO: 20:49:27.495:  --max-load    = '16' (default)
** INFO: 20:49:27.495:  --max-zoom    = '8' (user-specified/from config)
** INFO: 20:49:27.495:  --min-zoom    = '0' (user-specified/from config)
** INFO: 20:49:27.495:  --num-threads = '4' (user-specified/from config)
** INFO: 20:49:27.495:  --socket      = 'build/tests/run/file/renderd0.sock' 
(user-specified/from config)
** INFO: 20:49:27.495:  --tile-dir    = 'build/tests/tiles/file' 
(user-specified/from config)
** Message: 20:49:27.495: Starting 4 rendering threads
** Message: 20:49:27.495: Rendering all tiles from zoom 0 to zoom 8
** Message: 20:49:27.495: Rendering all tiles for zoom 0 from (0, 0) to (0, 0)
** Message: 20:49:27.495: Rendering all tiles for zoom 1 from (0, 0) to (0, 0)
** Message: 20:49:27.495: Rendering all tiles for zoom 2 from (0, 1) to (1, 1)
** Message: 20:49:27.495: Rendering all tiles for zoom 3 from (1, 2) to (2, 3)
** Message: 20:49:27.495: Rendering all tiles for zoom 4 from (2, 5) to (5, 6)
** Message: 20:49:27.495: Rendering all tiles for zoom 5 from (4, 10) to (10, 
13)
** Message: 20:49:27.495: Rendering all tiles for zoom 6 from (9, 21) to (20, 
27)
** Message: 20:49:27.526: Rendering all tiles for zoom 7 from (19, 43) to (40, 
55)
** Message: 20:49:28.565: Rendering all tiles for zoom 8 from (39, 87) to (80, 
110)
** Message: 20:49:35.567: Waiting for rendering threads to finish
** Message: 20:49:37.567: Total for all tiles rendered
** Message: 20:49:37.567: Metatiles rendered:
** Message: 20:49:37.567:       47 in 10.07 seconds (4.67/s)
** Message: 20:49:37.567: Total tiles rendered:
** Message: 20:49:37.567:       3008 in 10.07 seconds (298.64/s)
** Message: 20:49:37.567: Total tiles handled:
** Message: 20:49:37.567:       47 in 10.07 seconds (4.67/s)
```



As well as included some debug messaging to indicate what is being enqueued.
```
** Message: 09:20:42.038: Rendering all tiles from zoom 0 to zoom 8
** Message: 09:20:42.038: Rendering all tiles for zoom 0 from (0, 0) to (0, 0)
** Message: 09:20:42.038: Rendering all tiles for zoom 1 from (0, 0) to (0, 0)
** Message: 09:20:42.038: Rendering all tiles for zoom 2 from (0, 1) to (1, 1)
** (process:188843): DEBUG: 09:20:42.038: Enqueueing tile (xmlname=default x=0 
y=0 z=0)
** Message: 09:20:42.038: Rendering all tiles for zoom 3 from (1, 2) to (2, 3)
** (process:188843): DEBUG: 09:20:42.038: Enqueueing tile (xmlname=default x=0 
y=0 z=1)
** (process:188843): DEBUG: 09:20:42.038: Enqueueing tile (xmlname=default x=0 
y=0 z=2)
** (process:188843): DEBUG: 09:20:42.038: Enqueueing tile (xmlname=default x=0 
y=0 z=3)
** Message: 09:20:42.038: Rendering all tiles for zoom 4 from (2, 5) to (5, 6)
** (process:188843): DEBUG: 09:20:42.038: Enqueueing tile (xmlname=default x=0 
y=0 z=4)
** Message: 09:20:42.038: Rendering all tiles for zoom 5 from (4, 10) to (10, 
13)
** (process:188843): DEBUG: 09:20:42.038: Enqueueing tile (xmlname=default x=0 
y=8 z=5)
** (process:188843): DEBUG: 09:20:42.038: Enqueueing tile (xmlname=default x=8 
y=8 z=5)
** Message: 09:20:42.038: Rendering all tiles for zoom 6 from (9, 21) to (20, 
27)
** (process:188843): DEBUG: 09:20:42.038: Enqueueing tile (xmlname=default x=8 
y=16 z=6)
** (process:188843): DEBUG: 09:20:42.038: Enqueueing tile (xmlname=default x=8 
y=24 z=6)
** (process:188843): DEBUG: 09:20:42.055: Enqueueing tile (xmlname=default x=16 
y=16 z=6)
** (process:188843): DEBUG: 09:20:42.058: Enqueueing tile (xmlname=default x=16 
y=24 z=6)
** Message: 09:20:42.067: Rendering all tiles for zoom 7 from (19, 43) to (40, 
55)
** (process:188843): DEBUG: 09:20:42.067: Enqueueing tile (xmlname=default x=16 
y=40 z=7)
** (process:188843): DEBUG: 09:20:42.086: Enqueueing tile (xmlname=default x=16 
y=48 z=7)
** (process:188843): DEBUG: 09:20:42.094: Enqueueing tile (xmlname=default x=24 
y=40 z=7)
** (process:188843): DEBUG: 09:20:42.106: Enqueueing tile (xmlname=default x=24 
y=48 z=7)
** (process:188843): DEBUG: 09:20:42.106: Enqueueing tile (xmlname=default x=32 
y=40 z=7)
** (process:188843): DEBUG: 09:20:43.086: Enqueueing tile (xmlname=default x=32 
y=48 z=7)
** (process:188843): DEBUG: 09:20:43.094: Enqueueing tile (xmlname=default x=40 
y=40 z=7)
** (process:188843): DEBUG: 09:20:43.106: Enqueueing tile (xmlname=default x=40 
y=48 z=7)
** Message: 09:20:43.106: Rendering all tiles for zoom 8 from (39, 87) to (80, 
110)
** (process:188843): DEBUG: 09:20:43.106: Enqueueing tile (xmlname=default x=32 
y=80 z=8)
** (process:188843): DEBUG: 09:20:44.087: Enqueueing tile (xmlname=default x=32 
y=88 z=8)
** (process:188843): DEBUG: 09:20:44.095: Enqueueing tile (xmlname=default x=32 
y=96 z=8)
** (process:188843): DEBUG: 09:20:44.106: Enqueueing tile (xmlname=default x=32 
y=104 z=8)
** (process:188843): DEBUG: 09:20:44.106: Enqueueing tile (xmlname=default x=40 
y=80 z=8)
** (process:188843): DEBUG: 09:20:45.087: Enqueueing tile (xmlname=default x=40 
y=88 z=8)
** (process:188843): DEBUG: 09:20:45.095: Enqueueing tile (xmlname=default x=40 
y=96 z=8)
** (process:188843): DEBUG: 09:20:45.106: Enqueueing tile (xmlname=default x=40 
y=104 z=8)
** (process:188843): DEBUG: 09:20:45.106: Enqueueing tile (xmlname=default x=48 
y=80 z=8)
** (process:188843): DEBUG: 09:20:46.087: Enqueueing tile (xmlname=default x=48 
y=88 z=8)
** (process:188843): DEBUG: 09:20:46.095: Enqueueing tile (xmlname=default x=48 
y=96 z=8)
** (process:188843): DEBUG: 09:20:46.107: Enqueueing tile (xmlname=default x=48 
y=104 z=8)
** (process:188843): DEBUG: 09:20:46.107: Enqueueing tile (xmlname=default x=56 
y=80 z=8)
** (process:188843): DEBUG: 09:20:47.088: Enqueueing tile (xmlname=default x=56 
y=88 z=8)
** (process:188843): DEBUG: 09:20:47.096: Enqueueing tile (xmlname=default x=56 
y=96 z=8)
** (process:188843): DEBUG: 09:20:47.107: Enqueueing tile (xmlname=default x=56 
y=104 z=8)
** (process:188843): DEBUG: 09:20:47.107: Enqueueing tile (xmlname=default x=64 
y=80 z=8)
** (process:188843): DEBUG: 09:20:48.088: Enqueueing tile (xmlname=default x=64 
y=88 z=8)
** (process:188843): DEBUG: 09:20:48.096: Enqueueing tile (xmlname=default x=64 
y=96 z=8)
** (process:188843): DEBUG: 09:20:48.107: Enqueueing tile (xmlname=default x=64 
y=104 z=8)
** (process:188843): DEBUG: 09:20:48.107: Enqueueing tile (xmlname=default x=72 
y=80 z=8)
** (process:188843): DEBUG: 09:20:49.088: Enqueueing tile (xmlname=default x=72 
y=88 z=8)
** (process:188843): DEBUG: 09:20:49.096: Enqueueing tile (xmlname=default x=72 
y=96 z=8)
** (process:188843): DEBUG: 09:20:49.107: Enqueueing tile (xmlname=default x=72 
y=104 z=8)
** (process:188843): DEBUG: 09:20:49.107: Enqueueing tile (xmlname=default x=80 
y=80 z=8)
** (process:188843): DEBUG: 09:20:50.088: Enqueueing tile (xmlname=default x=80 
y=88 z=8)
** (process:188843): DEBUG: 09:20:50.096: Enqueueing tile (xmlname=default x=80 
y=96 z=8)
** (process:188843): DEBUG: 09:20:50.108: Enqueueing tile (xmlname=default x=80 
y=104 z=8)
```


-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/mod_tile/issues/501#issuecomment-5999544238
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/mod_tile/issues/501/[email protected]>
_______________________________________________
Tile-serving mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/tile-serving

Reply via email to