dbatomic commented on code in PR #44004:
URL: https://github.com/apache/spark/pull/44004#discussion_r1407438638


##########
sql/core/src/main/scala/org/apache/spark/sql/functions.scala:
##########
@@ -3269,14 +3269,6 @@ object functions {
    */
   def raise_error(c: Column): Column = Column.fn("raise_error", c)
 
-  /**
-   * Throws an exception with the provided error class and parameter map.
-   *
-   * @group misc_funcs
-   * @since 4.0.0
-   */
-  def raise_error(c: Column, e: Column): Column = Column.fn("raise_error", c, 
e)

Review Comment:
   done.
   
   @beliefer  Yes, idea is to forbid errorParams exposure, given that we will 
always throw USER_RAISED_EXCEPTION with single arg (string representing the 
message). @cloud-fan, please confirm if I got things right.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to