Datalevin doesn't play well with sci (through malli)

[jeroenvandijk]

Datalevin doesn't play well with sci (through malli)

[huahaiy]

As per discussions on slack, requiring people to have a binding to use Datalevin functions seems a bit excessive, so we will keep the set-config function proposal, and hope people who have similar issues will land it here.

[huahaiy]

Using print for regex is when importing/exporting data as text, so it is possible to only add the print-method when exporting.

[huahaiy]

Just limit the change of print-method to dumping the DB.

[huahaiy]

Because some data contains regex, in order to store and retrieve such data, print-method for regex was added, e.g. they are often used in transaction functions.

We can make this print-method optional for regex in order to support your use case. I can add a function set-confg to allow setting some global configurations. In this case, (set-config :store-regex? false) can be called to disable adding print-method for regex.

[jeroenvandijk]

I understand it is part of the serialization format. I would argue though that including the Datalevin namespace in your Clojure project should not break something that can be considered standard Clojure code. Especially for new users it would not be obvious to use your proposed setting. Also, what do you do when you need both. Maybe there is an alternative solution (idea below)?

Note: I haven't touched Datalevin much so I'm not sure about the impact of any change to this project.

;; in datalevin.bits

;; Don;t interfere with Clojure by default
(def :^dynamic *datalevin* false)

;; For Datalevin types *datalevin* is true
(defmethod print-method Indexable
  [^Indexable i, ^Writer w]
  (.write w "#datalevin/Indexable ")
  ;; This is Datalevin territory
  (binding [*datalevin* true]
    (.write w (pr-str (pr-indexable i)))))

;; Store original print-method
(defonce stdlib-print-method-pattern (get-method print-method java.util.regex.Pattern))

;; Depending on the *datalevin* value we will adapt printing for regex (and maybe others
(defmethod print-method Pattern
  [^Pattern p, ^Writer w]
  (if *datalevin*
    (doto w
      (.write "#datalevin/regex ")
      (.write "\"")
      (.write ^String (s/escape (.toString p) {\\ "\\\\"}))
      (.write "\""))
    ;; => Default Clojure's print behaviour here
    (stdlib-print-method-pattern p w)))

[den1k]

Because some data contains regex, in order to store and retrieve such data, print-method for regex was added, e.g. they are often used in transaction functions.

is this because pr-str is used to write this data? Would it help to use custom functions similar to nippy's serialize?