Remove unnecessary conversion steps in PartiQLValueIonReaderBuilder (#…

…1456)
partiql · May 7, 2024 · 2a13661 · 2a13661 · github-actions · May 8, 2024
1 parent 4851cac
commit 2a13661
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/partiql-types/src/main/kotlin/org/partiql/value/io/PartiQLValueIonReader.kt b/partiql-types/src/main/kotlin/org/partiql/value/io/PartiQLValueIonReader.kt
@@ -3,10 +3,8 @@ package org.partiql.value.io
 import com.amazon.ion.IonReader
 import com.amazon.ion.IonType
 import com.amazon.ion.system.IonReaderBuilder
-import com.amazon.ion.system.IonSystemBuilder
 import com.amazon.ion.system.IonTextWriterBuilder
 import com.amazon.ionelement.api.IonElement
-import com.amazon.ionelement.api.toIonValue
 import org.partiql.value.DecimalValue
 import org.partiql.value.IntValue
 import org.partiql.value.PartiQLValue
@@ -626,9 +624,8 @@ public class PartiQLValueIonReaderBuilder private constructor(
 
     public fun build(ionElement: IonElement): PartiQLValueReader {
         val out = ByteArrayOutputStream()
-        val reader = IonReaderBuilder.standard().build(ionElement.toIonValue(IonSystemBuilder.standard().build()))
         val writer = IonTextWriterBuilder.standard().build(out)
-        writer.writeValues(reader)
+        ionElement.writeTo(writer)
         val input = ByteArrayInputStream(out.toByteArray())
         return build(input)
     }