Clojurians Log v2
Clojure programming
Channels
Apps
pedestal 2024-10-09
hlship 2024-10-09T21:27:55.890589Z
It took some experimentation, but the CVE checks are now re-enabled on CI builds.
🎉 2
hlship 2024-10-09T23:45:02.444809Z
;┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━┳━━━━━━━━━━━━━┓
;┃ Expression ┃ Mean ┃ Var ┃
;┣━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━╋━━━━━━━━━━━━━┫
;┃ (execute-static :small :map-tree) ┃ 18.05 µs ┃ ± 258.02 ns ┃ (fastest)
;┃ (execute-static :small :sawtooth) ┃ 21.48 µs ┃ ± 695.38 ns ┃
;┃ (execute-static :medium :map-tree) ┃ 176.26 µs ┃ ± 1.92 µs ┃
;┃ (execute-static :medium :sawtooth) ┃ 202.23 µs ┃ ± 4.14 µs ┃
;┃ (execute-static :large :map-tree) ┃ 16.99 ms ┃ ± 181.81 µs ┃
;┃ (execute-static :large :sawtooth) ┃ 19.56 ms ┃ ± 395.42 µs ┃ (slowest)
;┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┻━━━━━━━━━━━┻━━━━━━━━━━━━━┛hlship 2024-10-09T23:47:38.012629Z
Starting to optimize :sawtooth for static-only (no path params) routing tables. Already it is quick enough, in my opinion, to be the default router.
👍 3
souenzzo 2024-10-10T16:59:13.481669Z
i think that the defualt router should be the best on error report//dev experience, not the fastest
hlship 2024-10-10T17:29:41.435499Z
My only concern is that upgrades from 0.7 to 0.8 may see different behavior with Sawtooth than with prefix-tree, but you would only see that if there were route conflicts to begin with, and (assuming there are tests) you'll see the warnings and know something is up immediately, and can decide to remove conflicts or move back to :prefix-tree.
👍 1