Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12708#discussion_r61755088
--- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/Row.scala ---
@@ -335,6 +358,19 @@ trait Row extends Serializable {
def getAs[T](fieldName: String): T = getAs[T](fieldIndex(fieldName))
/**
+ * Returns the value of a given fieldName as a scala.util.Try object.
+ */
+ def attempt[T](fieldName: String): Try[T] =
Try(getAs[T](fieldIndex(fieldName)))
--- End diff --
This part of the code isn't my department, so take this with a grain of
salt. I'm not sure what to make of `attempt` as an API method. (I know from the
discussion what it's for.) I'd expect to see getters, or maybe getters that
return an `Option`. There's already a method that gets a value or `null`, which
can be made into an `Option` by the caller. Right? or else I miss what else
it's for. I assume this is kind of mimicking the JDBC API which would return
nulls for nullable values.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
