You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/07/20 06:33:25 UTC

[GitHub] [spark] beliefer opened a new pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

beliefer opened a new pull request #29161:
URL: https://github.com/apache/spark/pull/29161


   ### What changes were proposed in this pull request?
   The current implement of regexp_extract will throws a unprocessed exception show below:
   SELECT regexp_extract('1a 2b 14m', 'd+' -1)
   
   ```
   java.lang.IndexOutOfBoundsException: No group -1
   java.util.regex.Matcher.group(Matcher.java:538)
   org.apache.spark.sql.catalyst.expressions.RegExpExtract.nullSafeEval(regexpExpressions.scala:455)
   org.apache.spark.sql.catalyst.expressions.TernaryExpression.eval(Expression.scala:704)
   org.apache.spark.sql.catalyst.optimizer.ConstantFolding$$anonfun$apply$1$$anonfun$applyOrElse$1.applyOrElse(expressions.scala:52)
   org.apache.spark.sql.catalyst.optimizer.ConstantFolding$$anonfun$apply$1$$anonfun$applyOrElse$1.applyOrElse(expressions.scala:45)
   ```
   
   
   ### Why are the changes needed?
   Fix a bug `java.lang.IndexOutOfBoundsException: No group -1`
   
   
   ### Does this PR introduce _any_ user-facing change?
   Yes
   
   
   ### How was this patch tested?
   new UT
   


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661056431


   **[Test build #126172 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126172/testReport)** for PR 29161 at commit [`fd0d54c`](https://github.com/apache/spark/commit/fd0d54c5d82d25249856882f03289a1c3c7d596c).
    * This patch **fails PySpark pip packaging tests**.
    * This patch merges cleanly.
    * This patch adds no public classes.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660833344


   **[Test build #126156 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126156/testReport)** for PR 29161 at commit [`9ab5189`](https://github.com/apache/spark/commit/9ab5189ca58004261ec43156f3bb516d5bf544a1).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA removed a comment on pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661565693


   **[Test build #126210 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126210/testReport)** for PR 29161 at commit [`6d6008e`](https://github.com/apache/spark/commit/6d6008e42bea39a16f7b370d0d7cdbf067c878d6).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660845129






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660894821


   **[Test build #126171 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126171/testReport)** for PR 29161 at commit [`f68e5c3`](https://github.com/apache/spark/commit/f68e5c3ccbefd362019f5717ce5f7a3a36aae0d3).
    * This patch **fails Scala style tests**.
    * This patch merges cleanly.
    * This patch adds no public classes.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661057228


   Test FAILed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126172/
   Test FAILed.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660845025






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661318966


   Merged build finished. Test FAILed.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660897869


   **[Test build #126172 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126172/testReport)** for PR 29161 at commit [`fd0d54c`](https://github.com/apache/spark/commit/fd0d54c5d82d25249856882f03289a1c3c7d596c).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661125834






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660893983


   **[Test build #126171 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126171/testReport)** for PR 29161 at commit [`f68e5c3`](https://github.com/apache/spark/commit/f68e5c3ccbefd362019f5717ce5f7a3a36aae0d3).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661057217


   Merged build finished. Test FAILed.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661057217






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660845129






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660833871






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660894710






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660894839






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA removed a comment on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660839264






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661566494






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661125180


   **[Test build #126188 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126188/testReport)** for PR 29161 at commit [`fd0d54c`](https://github.com/apache/spark/commit/fd0d54c5d82d25249856882f03289a1c3c7d596c).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660898651






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661565693


   **[Test build #126210 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126210/testReport)** for PR 29161 at commit [`6d6008e`](https://github.com/apache/spark/commit/6d6008e42bea39a16f7b370d0d7cdbf067c878d6).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661676766


   Merged build finished. Test FAILed.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660839745






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] beliefer commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
beliefer commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661124321


   retest this please


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660897869


   **[Test build #126172 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126172/testReport)** for PR 29161 at commit [`fd0d54c`](https://github.com/apache/spark/commit/fd0d54c5d82d25249856882f03289a1c3c7d596c).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661676771


   Test FAILed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126210/
   Test FAILed.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661676243


   **[Test build #126210 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126210/testReport)** for PR 29161 at commit [`6d6008e`](https://github.com/apache/spark/commit/6d6008e42bea39a16f7b370d0d7cdbf067c878d6).
    * This patch **fails due to an unknown error code, -9**.
    * This patch merges cleanly.
    * This patch adds no public classes.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] maropu commented on a change in pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
maropu commented on a change in pull request #29161:
URL: https://github.com/apache/spark/pull/29161#discussion_r457099324



##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
##########
@@ -412,7 +412,10 @@ case class RegExpReplace(subject: Expression, regexp: Expression, rep: Expressio
 
 object RegExpExtract {
   def checkGroupIndex(groupCount: Int, groupIndex: Int): Unit = {
-    if (groupCount < groupIndex) {
+    if (groupIndex < 0) {
+      throw new IllegalArgumentException(
+        s"The specified group index cannot be less than zero")

Review comment:
       nit: remove `s` in the head.




----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661318966






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] beliefer commented on a change in pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
beliefer commented on a change in pull request #29161:
URL: https://github.com/apache/spark/pull/29161#discussion_r457792887



##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
##########
@@ -426,6 +428,14 @@ object RegExpExtract {
  */
 @ExpressionDescription(
   usage = "_FUNC_(str, regexp[, idx]) - Extracts a group that matches `regexp`.",
+  arguments = """
+    Arguments:
+      * str - a string expression.
+      * regexp - a string expression. a string representing a regular expression.
+          The regex string should be a Java regular expression.
+      * idx - a int expression that representing the group index. The group index should be
+          0 or positive. If `idx` is not specified, the default group index value is 1.

Review comment:
       OK.




----------------------------------------------------------------
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.

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



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

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
dongjoon-hyun commented on a change in pull request #29161:
URL: https://github.com/apache/spark/pull/29161#discussion_r457690165



##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
##########
@@ -426,6 +428,14 @@ object RegExpExtract {
  */
 @ExpressionDescription(
   usage = "_FUNC_(str, regexp[, idx]) - Extracts a group that matches `regexp`.",
+  arguments = """
+    Arguments:
+      * str - a string expression.
+      * regexp - a string expression. a string representing a regular expression.

Review comment:
       Shall we remove the redundant `a string expression`?
   ```
   - * regexp - a string expression. a string representing a regular expression.
   + * regexp - a string representing a regular expression.
   ```




----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660833871






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
dongjoon-hyun commented on a change in pull request #29161:
URL: https://github.com/apache/spark/pull/29161#discussion_r457690897



##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
##########
@@ -426,6 +428,14 @@ object RegExpExtract {
  */
 @ExpressionDescription(
   usage = "_FUNC_(str, regexp[, idx]) - Extracts a group that matches `regexp`.",
+  arguments = """
+    Arguments:
+      * str - a string expression.
+      * regexp - a string expression. a string representing a regular expression.
+          The regex string should be a Java regular expression.
+      * idx - a int expression that representing the group index. The group index should be
+          0 or positive. If `idx` is not specified, the default group index value is 1.

Review comment:
       `or or positive` -> `non-negative`?




----------------------------------------------------------------
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.

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



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

[GitHub] [spark] beliefer commented on a change in pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
beliefer commented on a change in pull request #29161:
URL: https://github.com/apache/spark/pull/29161#discussion_r457792805



##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
##########
@@ -426,6 +428,14 @@ object RegExpExtract {
  */
 @ExpressionDescription(
   usage = "_FUNC_(str, regexp[, idx]) - Extracts a group that matches `regexp`.",
+  arguments = """
+    Arguments:
+      * str - a string expression.
+      * regexp - a string expression. a string representing a regular expression.
+          The regex string should be a Java regular expression.
+      * idx - a int expression that representing the group index. The group index should be

Review comment:
       Thanks.




----------------------------------------------------------------
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.

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



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

[GitHub] [spark] maropu commented on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
maropu commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660858398


   Nice catch, @beliefer . For better user's documents, could you add `arguments` in `ExpressionDescription` of `RegExpExtract` ? I think we could describe `index should be 0 or positive` there.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661566494






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] beliefer commented on a change in pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
beliefer commented on a change in pull request #29161:
URL: https://github.com/apache/spark/pull/29161#discussion_r457792605



##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
##########
@@ -426,6 +428,14 @@ object RegExpExtract {
  */
 @ExpressionDescription(
   usage = "_FUNC_(str, regexp[, idx]) - Extracts a group that matches `regexp`.",
+  arguments = """
+    Arguments:
+      * str - a string expression.
+      * regexp - a string expression. a string representing a regular expression.

Review comment:
       OK.




----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660894710






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660894849


   Test FAILed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126171/
   Test FAILed.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] dongjoon-hyun closed pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
dongjoon-hyun closed pull request #29161:
URL: https://github.com/apache/spark/pull/29161


   


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] beliefer commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
beliefer commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660891690


   @maropu OK.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660893983


   **[Test build #126171 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126171/testReport)** for PR 29161 at commit [`f68e5c3`](https://github.com/apache/spark/commit/f68e5c3ccbefd362019f5717ce5f7a3a36aae0d3).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661125180


   **[Test build #126188 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126188/testReport)** for PR 29161 at commit [`fd0d54c`](https://github.com/apache/spark/commit/fd0d54c5d82d25249856882f03289a1c3c7d596c).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661318293


   **[Test build #126188 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126188/testReport)** for PR 29161 at commit [`fd0d54c`](https://github.com/apache/spark/commit/fd0d54c5d82d25249856882f03289a1c3c7d596c).
    * This patch **fails PySpark pip packaging tests**.
    * This patch merges cleanly.
    * This patch adds no public classes.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660839745






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] beliefer commented on pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
beliefer commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661611000


   @dongjoon-hyun @maropu Thanks for your help!


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
dongjoon-hyun commented on a change in pull request #29161:
URL: https://github.com/apache/spark/pull/29161#discussion_r457690531



##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
##########
@@ -426,6 +428,14 @@ object RegExpExtract {
  */
 @ExpressionDescription(
   usage = "_FUNC_(str, regexp[, idx]) - Extracts a group that matches `regexp`.",
+  arguments = """
+    Arguments:
+      * str - a string expression.
+      * regexp - a string expression. a string representing a regular expression.
+          The regex string should be a Java regular expression.
+      * idx - a int expression that representing the group index. The group index should be

Review comment:
       `a int` -> `an integer`?




----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660898651






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL] Add a boundary condition for negative index in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661676766






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins commented on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661125834






----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660845195


   Test FAILed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126157/
   Test FAILed.


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] beliefer commented on a change in pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
beliefer commented on a change in pull request #29161:
URL: https://github.com/apache/spark/pull/29161#discussion_r457104207



##########
File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
##########
@@ -412,7 +412,10 @@ case class RegExpReplace(subject: Expression, regexp: Expression, rep: Expressio
 
 object RegExpExtract {
   def checkGroupIndex(groupCount: Int, groupIndex: Int): Unit = {
-    if (groupCount < groupIndex) {
+    if (groupIndex < 0) {
+      throw new IllegalArgumentException(
+        s"The specified group index cannot be less than zero")

Review comment:
       OK. Thanks for your remind.




----------------------------------------------------------------
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.

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



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

[GitHub] [spark] SparkQA commented on pull request #29161: [SPARK-32365][SQL]Fix java.lang.IndexOutOfBoundsException: No group -1

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-660839264


   **[Test build #126157 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126157/testReport)** for PR 29161 at commit [`e31136c`](https://github.com/apache/spark/commit/e31136c6e5c205892d50e5d4cbfff7c591cd411f).


----------------------------------------------------------------
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.

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



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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29161: [SPARK-32365][SQL]Fix IndexOutOfBoundsException in regexp_extract

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #29161:
URL: https://github.com/apache/spark/pull/29161#issuecomment-661318972


   Test FAILed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126188/
   Test FAILed.


----------------------------------------------------------------
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.

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



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