Added formatter dependency to samples, turn warning into an error since the default value is false

Author: gavlyukovskiy

README.md

@@ -102,6 +102,8 @@ public JdbcEventListener myListener() {
 This done by adding `RuntimeListenerSupportFactory` into P6Spy `modulelist`, overriding this property will cause to not registering factory thus listeners will not be applied
 
 You can configure small set of parameters in your `application.properties`:
+> [!NOTE]
+> Configuration below indicates al possible parameters together with their default values and **does not** need to be set explicitly
 ```properties
 # Register P6LogFactory to log JDBC events
 decorator.datasource.p6spy.enable-logging=true
@@ -180,6 +182,8 @@ public ConnectionIdManagerProvider connectionIdManagerProvider() {
 }
 ```
 You can configure logging, query/slow query listeners and more using your `application.properties`:
+> [!NOTE]
+> Configuration below indicates al possible parameters together with their default values and **does not** need to be set explicitly
 ```properties
 # One of logging libraries (slf4j, jul, common, sysout)
 decorator.datasource.datasource-proxy.logging=slf4j
@@ -199,7 +203,7 @@ decorator.datasource.datasource-proxy.multiline=true
 
 # Formats the SQL for better readability. Uses Hibernate's formatter if present on the class path. If you opted in for a different JPA provider you need to add https://github.com/vertical-blank/sql-formatter as a runtime dependency to your app  to enable this. 
 # Mutually exclusive with json-format=true
-decorator.datasource.datasource-proxy.format-sql=true
+decorator.datasource.datasource-proxy.format-sql=false
 decorator.datasource.datasource-proxy.json-format=false
 
 # Enable Query Metrics
@@ -234,6 +238,8 @@ All beans of type `ConnectionAcquiringStrategyFactory` are used to provide `Conn
 `EventListener<? extends Event>` beans can be registered to subscribe on events of flexy-pool (e.g. `ConnectionAcquireTimeThresholdExceededEvent`, `ConnectionLeaseTimeThresholdExceededEvent`).
 
 You can configure your `FlexyPoolDataSource` by using bean `FlexyPoolConfigurationBuilderCustomizer` or properties:
+> [!NOTE]
+> Configuration below indicates al possible parameters together with their default values and **does not** need to be set explicitly
 ```properties
 # Increments pool size if connection acquire request has timed out
 decorator.datasource.flexy-pool.acquiring-strategy.increment-pool.max-overflow-pool-size=15

datasource-decorator-spring-boot-autoconfigure/src/main/java/com/github/gavlyukovskiy/boot/jdbc/decorator/dsproxy/ProxyDataSourceBuilderConfigurer.java

@@ -132,7 +132,7 @@ public void configure(ProxyDataSourceBuilder proxyDataSourceBuilder, DataSourceP
             if (formatQueryCallback != null) {
                 proxyDataSourceBuilder.formatQuery(formatQueryCallback);
             } else {
-                log.warn("formatSql requested but cannot be enabled because no formatter is present (neither Hibernate nor SqlFormatter).");
+                throw new IllegalStateException("'datasource-proxy.format-sql' was set to 'true', but cannot be enabled because no formatter is present in the classpath (neither 'org.hibernate:hibernate-core' nor 'com.github.vertical-blank:sql-formatter').");
             }
         }
 

samples/datasource-proxy-sample/build.gradle.kts

@@ -23,6 +23,7 @@ dependencies {
 
     implementation("com.h2database:h2")
     implementation("org.apache.commons:commons-io:1.3.2")
+    implementation("com.github.vertical-blank:sql-formatter:2.0.4")
 
     testImplementation("org.springframework.boot:spring-boot-starter-test")
 }

samples/datasource-proxy-sample/src/main/resources/application.yml

@@ -19,3 +19,9 @@ spring:
     init:
       mode: never
 
+decorator:
+  datasource:
+    datasource-proxy:
+      format-sql: true
+      query:
+        log-level: INFO