Removed UpdateException

Author: Jolanrensen

core/src/main/kotlin/org/jetbrains/kotlinx/dataframe/impl/api/update.kt

@@ -23,7 +23,6 @@ import org.jetbrains.kotlinx.dataframe.api.with
 import org.jetbrains.kotlinx.dataframe.columns.ColumnGroup
 import org.jetbrains.kotlinx.dataframe.columns.FrameColumn
 import org.jetbrains.kotlinx.dataframe.columns.size
-import org.jetbrains.kotlinx.dataframe.exceptions.DataFrameError
 import org.jetbrains.kotlinx.dataframe.impl.columns.AddDataRowImpl
 import org.jetbrains.kotlinx.dataframe.impl.createDataCollector
 import org.jetbrains.kotlinx.dataframe.index
@@ -95,10 +94,6 @@ private fun <C, R> ColumnGroup<C>.replaceRowsIf(
         .asColumnGroup()
         .cast()
 
-public class UpdateException(override val message: String, cause: Throwable? = null) :
-    IllegalStateException(message, cause),
-    DataFrameError
-
 internal fun <T, C> DataColumn<C>.updateImpl(
     df: DataFrame<T>,
     filter: RowValueFilter<T, C>?,
@@ -123,7 +118,7 @@ internal fun <T, C> DataColumn<C>.updateImpl(
         }
         return collector.toColumn(src.name).cast()
     } catch (e: Throwable) {
-        throw UpdateException("Could not update column '${src.name}': ${e.message}", e)
+        throw IllegalStateException("Could not update column '${src.name}': ${e.message}", e)
     }
 }
 

core/src/test/kotlin/org/jetbrains/kotlinx/dataframe/api/update.kt

@@ -5,7 +5,6 @@ import io.kotest.matchers.shouldBe
 import org.jetbrains.kotlinx.dataframe.DataColumn
 import org.jetbrains.kotlinx.dataframe.DataFrame
 import org.jetbrains.kotlinx.dataframe.annotations.DataSchema
-import org.jetbrains.kotlinx.dataframe.impl.api.UpdateException
 import org.jetbrains.kotlinx.dataframe.impl.nothingType
 import org.jetbrains.kotlinx.dataframe.impl.nullableNothingType
 import org.jetbrains.kotlinx.dataframe.size
@@ -101,7 +100,7 @@ class UpdateTests {
             .schema() shouldBe nullFilledDf.schema()
 
         // or 'Nothing', aka, return early/throw exception
-        shouldThrow<UpdateException> {
+        shouldThrow<IllegalStateException> {
             nullFilledDf.update { "a"<Nothing?>() }.with { error("Nothing") }
         }.cause!!.message shouldBe "Nothing"
     }

core/src/test/kotlin/org/jetbrains/kotlinx/dataframe/testSets/person/DataFrameTests.kt

@@ -168,7 +168,6 @@ import org.jetbrains.kotlinx.dataframe.get
 import org.jetbrains.kotlinx.dataframe.hasNulls
 import org.jetbrains.kotlinx.dataframe.impl.DataFrameImpl
 import org.jetbrains.kotlinx.dataframe.impl.DataFrameSize
-import org.jetbrains.kotlinx.dataframe.impl.api.UpdateException
 import org.jetbrains.kotlinx.dataframe.impl.api.convertToImpl
 import org.jetbrains.kotlinx.dataframe.impl.between
 import org.jetbrains.kotlinx.dataframe.impl.columns.isMissingColumn
@@ -1713,7 +1712,7 @@ class DataFrameTests : BaseTest() {
         df.update("name").at(0).with { "ALICE" }
     }
 
-    @Test(expected = UpdateException::class)
+    @Test(expected = IllegalStateException::class)
     fun `update with wrong type`() {
         typed.update("age").with { "string" }
     }