Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

Commit 210ce3c

Browse files
authored
[clojure] Create new Fortune object in scope of request (#10656)
The new fortune object must be constructed in the scope of the request, not as a constant: > Within the scope of the request, a new Fortune object must be constructed and added to the list. https://github.com/TechEmpower/FrameworkBenchmarks/wiki/Project-Information-Framework-Tests-Overview#fortunes
1 parent 08a69ec commit 210ce3c

2 files changed

Lines changed: 6 additions & 7 deletions

File tree

frameworks/Clojure/ring-http-exchange/src/ring_http_exchange/input_stream_handler.clj

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,18 +10,17 @@
1010
(def query-fortunes (boa/build-query (boa/->NextJdbcAdapter) "fortune.sql"))
1111

1212
(def ^:private hello-world-bytes (.getBytes "Hello, World!"))
13-
(def ^:private ^:const additional-message {:id 0
14-
:message "Additional fortune added at request time."})
13+
1514
(def ^:private ^:const fortune-headers {"Server" "ring-http-exchange"
1615
"Content-Type" "text/html; charset=UTF-8"})
1716

18-
1917
(def ^:private render-fortune (majavat/build-html-renderer "fortune.html"
2018
{:renderer (->InputStreamRenderer)}))
2119

2220
(defn- get-body [datasource]
2321
(let [context (as-> (query-fortunes datasource) fortunes
24-
(conj fortunes additional-message)
22+
(conj fortunes {:id 0
23+
:message "Additional fortune added at request time."})
2524
(sort-by :message fortunes))]
2625
(render-fortune {:messages context})))
2726

frameworks/Clojure/ring-http-exchange/src/ring_http_exchange/string_handler.clj

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,7 @@
99
(def query-fortunes (boa/build-query (boa/->NextJdbcAdapter) "fortune.sql"))
1010

1111
(def ^:private ^:const hello-world "Hello, World!")
12-
(def ^:private ^:const additional-message {:id 0
13-
:message "Additional fortune added at request time."})
12+
1413
(def ^:private ^:const fortune-headers {"Server" "ring-http-exchange"
1514
"Content-Type" "text/html; charset=UTF-8"})
1615
(def ^:private ^:const json-headers {"Server" "ring-http-exchange"
@@ -22,7 +21,8 @@
2221

2322
(defn- get-body [datasource]
2423
(let [context (as-> (query-fortunes datasource) fortunes
25-
(conj fortunes additional-message)
24+
(conj fortunes {:id 0
25+
:message "Additional fortune added at request time."})
2626
(sort-by :message fortunes))]
2727
(render-fortune {:messages context})))
2828

0 commit comments

Comments
 (0)