feat!: support dataProperty with ParallelRequestExecutor

Adds proper handling for `dataProperties` to `ParallelRequestExecutor`
and gets rid of `DataLoaderPreparedRequestExecutor`, along with the
option to configure the request executor in `SchemaConfiguration` and
related `DataLoaderPreparedRequestExecutor`-specific options.

This also consolidates some tests that were hardcoded to use one of
the request executors, and therefore missed the other, and extends
coverage for dataProperties a bit. Further, this reduces visibility of
some functions to `internal` to reduce namespace pollution.

Resolves #58
Resolves #268

BREAKING CHANGE: The option to configure request executors has been
removed, and execution will now always use the `ParallelRequestExecutor`.
BREAKING CHANGE: The option to configure a timeout has been removed,
as it was only respected by `DataLoaderPreparedRequestExecutor`.
BREAKING CHANGE: The possibility to pass `ExecutionOptions` to schema
execution has been removed, as it was only respected by `DataLoaderPreparedRequestExecutor`.
BREAKING CHANGE: The `DataLoaderPreparedRequestExecutor`, `Executor`,
and `ExecutionOptions` classes have been removed.

Author: stuebingerb

docs/content/Reference/Type System/objects-and-interfaces.md

@@ -324,22 +324,11 @@ people in the second query. That's what the `dataProperty` will solve for you.
 
 ### Setting up
 
-Using DataLoaders requires the `DataLoaderPrepared` executor:
-
-```kotlin
-configure {
-    executor = Executor.DataLoaderPrepared
-}
-```
-
 === "Example"
     ```kotlin
     data class Person(val id: Int, val name: String)
     val people = (1..5).map { Person(it, "Name-$it") }
     ...
-    configure {
-        executor = Executor.DataLoaderPrepared
-    }
     query("people") {
         resolver { -> people }
     }
@@ -403,13 +392,3 @@ configure {
       }
     }
     ```
-
-### Current known issues
-
-This feature can be used in production but does currently have some issues:
-
-1. The `useDefaultPrettyPrint` doesn't work
-1. Order of fields are not guaranteed to match the order that was requested
-1. Custom generic type resolvers are not supported
-1. Schema stitching is not supported
-1. Other than that it should work as expected

docs/content/Reference/configuration.md

@@ -8,10 +8,8 @@ KGraphQL schema allows configuration of the following properties:
 | objectMapper                   | Schema is using Jackson ObjectMapper from this property                                                               | result of `jacksonObjectMapper()` from [jackson-kotlin-module](https://github.com/FasterXML/jackson-module-kotlin)                                                       |
 | acceptSingleValueAsArray       | Schema accepts single argument values as singleton list                                                               | `true`                                                                                                                                                                   |
 | coroutineDispatcher            | Schema is using CoroutineDispatcher from this property                                                                | [CommonPool](https://github.com/Kotlin/kotlinx.coroutines/blob/master/kotlinx-coroutines-core/src/main/kotlin/kotlinx/coroutines/experimental/CommonPool.kt)             |
-| executor                       |                                                                                                                       | [Executor.Parallel](https://github.com/aPureBase/KGraphQL/blob/master/kgraphql/src/main/kotlin/com/apurebase/kgraphql/schema/execution/Executor.kt)                      |
 | genericTypeResolver            | Schema is using generic type resolver from this property                                                              | [GenericTypeResolver.DEFAULT](https://github.com/aPureBase/KGraphQL/blob/master/kgraphql/src/main/kotlin/com/apurebase/kgraphql/schema/execution/GenericTypeResolver.kt) |
 | wrapErrors                     |                                                                                                                       | `true`                                                                                                                                                                   |
-| timeout                        |                                                                                                                       | `null`                                                                                                                                                                   |
 | introspection                  | Schema allows introspection (also affects SDL). Introspection can be disabled in production to reduce attack surface. | `true`                                                                                                                                                                   |
 
 === "Example"

gradle/libs.versions.toml

@@ -27,7 +27,6 @@ kotlinx-serialization-json = { module = "org.jetbrains.kotlinx:kotlinx-serializa
 jackson-core-databind = { module = "com.fasterxml.jackson.core:jackson-databind", version.ref = "jackson" }
 jackson-module-kotlin = { module = "com.fasterxml.jackson.module:jackson-module-kotlin", version.ref = "jackson" }
 aedile = { module = "com.sksamuel.aedile:aedile-core", version = "3.0.1" }
-deferredJsonBuilder = { module = "com.apurebase:DeferredJsonBuilder", version = "1.0.0" }
 ktor-client-core = { module = "io.ktor:ktor-client-core", version.ref = "ktor" }
 ktor-client-cio = { module = "io.ktor:ktor-client-cio", version.ref = "ktor" }
 ktor-server-core = { module = "io.ktor:ktor-server-core", version.ref = "ktor" }

kgraphql-ktor-stitched/api/kgraphql-ktor-stitched.api

@@ -60,8 +60,8 @@ public final class com/apurebase/kgraphql/stitched/StitchedKGraphQL {
 }
 
 public final class com/apurebase/kgraphql/stitched/schema/configuration/StitchedSchemaConfiguration : com/apurebase/kgraphql/configuration/SchemaConfiguration {
-	public fun <init> (ZJLcom/fasterxml/jackson/databind/ObjectMapper;ZLkotlinx/coroutines/CoroutineDispatcher;ZLcom/apurebase/kgraphql/schema/execution/Executor;Ljava/lang/Long;ZLcom/apurebase/kgraphql/schema/execution/GenericTypeResolver;Lcom/apurebase/kgraphql/schema/execution/ArgumentTransformer;Lcom/apurebase/kgraphql/stitched/schema/execution/RemoteRequestExecutor;Ljava/lang/String;)V
-	public synthetic fun <init> (ZJLcom/fasterxml/jackson/databind/ObjectMapper;ZLkotlinx/coroutines/CoroutineDispatcher;ZLcom/apurebase/kgraphql/schema/execution/Executor;Ljava/lang/Long;ZLcom/apurebase/kgraphql/schema/execution/GenericTypeResolver;Lcom/apurebase/kgraphql/schema/execution/ArgumentTransformer;Lcom/apurebase/kgraphql/stitched/schema/execution/RemoteRequestExecutor;Ljava/lang/String;ILkotlin/jvm/internal/DefaultConstructorMarker;)V
+	public fun <init> (ZJLcom/fasterxml/jackson/databind/ObjectMapper;ZLkotlinx/coroutines/CoroutineDispatcher;ZZLcom/apurebase/kgraphql/schema/execution/GenericTypeResolver;Lcom/apurebase/kgraphql/schema/execution/ArgumentTransformer;Lcom/apurebase/kgraphql/stitched/schema/execution/RemoteRequestExecutor;Ljava/lang/String;)V
+	public synthetic fun <init> (ZJLcom/fasterxml/jackson/databind/ObjectMapper;ZLkotlinx/coroutines/CoroutineDispatcher;ZZLcom/apurebase/kgraphql/schema/execution/GenericTypeResolver;Lcom/apurebase/kgraphql/schema/execution/ArgumentTransformer;Lcom/apurebase/kgraphql/stitched/schema/execution/RemoteRequestExecutor;Ljava/lang/String;ILkotlin/jvm/internal/DefaultConstructorMarker;)V
 	public final fun getLocalUrl ()Ljava/lang/String;
 	public final fun getRemoteExecutor ()Lcom/apurebase/kgraphql/stitched/schema/execution/RemoteRequestExecutor;
 }

kgraphql-ktor-stitched/src/main/kotlin/com/apurebase/kgraphql/stitched/schema/configuration/StitchedSchemaConfiguration.kt

@@ -3,7 +3,6 @@ package com.apurebase.kgraphql.stitched.schema.configuration
 import com.apurebase.kgraphql.ExperimentalAPI
 import com.apurebase.kgraphql.configuration.SchemaConfiguration
 import com.apurebase.kgraphql.schema.execution.ArgumentTransformer
-import com.apurebase.kgraphql.schema.execution.Executor
 import com.apurebase.kgraphql.schema.execution.GenericTypeResolver
 import com.apurebase.kgraphql.stitched.schema.execution.RemoteRequestExecutor
 import com.fasterxml.jackson.databind.ObjectMapper
@@ -20,8 +19,6 @@ class StitchedSchemaConfiguration(
     // execution
     coroutineDispatcher: CoroutineDispatcher,
     wrapErrors: Boolean,
-    executor: Executor,
-    timeout: Long?,
     // allow schema introspection
     introspection: Boolean = true,
     genericTypeResolver: GenericTypeResolver,
@@ -35,8 +32,6 @@ class StitchedSchemaConfiguration(
     useDefaultPrettyPrinter,
     coroutineDispatcher,
     wrapErrors,
-    executor,
-    timeout,
     introspection,
     genericTypeResolver,
     argumentTransformer

kgraphql-ktor-stitched/src/main/kotlin/com/apurebase/kgraphql/stitched/schema/dsl/StitchedSchemaConfigurationDSL.kt

@@ -24,8 +24,6 @@ open class StitchedSchemaConfigurationDSL : SchemaConfigurationDSL() {
             useDefaultPrettyPrinter = useDefaultPrettyPrinter,
             coroutineDispatcher = coroutineDispatcher,
             wrapErrors = wrapErrors,
-            executor = executor,
-            timeout = timeout,
             introspection = introspection,
             genericTypeResolver = genericTypeResolver,
             argumentTransformer = RemoteArgumentTransformer(),