You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by GitBox <gi...@apache.org> on 2022/08/25 07:45:52 UTC

[GitHub] [incubator-kyuubi] cxzl25 opened a new pull request, #3336: Use StageAttempt instead of StageId in SQLOperationListener

cxzl25 opened a new pull request, #3336:
URL: https://github.com/apache/incubator-kyuubi/pull/3336

   ### _Why are the changes needed?_
   Now `activeStages` only has the stageid, and does not record and output the number of stage retries, which may cause errors when the stage fails to retry.
   
   ### _How was this patch tested?_
   - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
   
   - [ ] Add screenshots for manual tests if appropriate
   
   - [ ] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request
   


-- 
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: notifications-unsubscribe@kyuubi.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


[GitHub] [incubator-kyuubi] pan3793 commented on pull request #3336: Use StageAttempt instead of StageId in SQLOperationListener

Posted by GitBox <gi...@apache.org>.
pan3793 commented on PR #3336:
URL: https://github.com/apache/incubator-kyuubi/pull/3336#issuecomment-1227992187

   Thanks, merging to master/1.6, also cc @SteNicholas 


-- 
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: notifications-unsubscribe@kyuubi.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


[GitHub] [incubator-kyuubi] pan3793 closed pull request #3336: Use StageAttempt instead of StageId in SQLOperationListener

Posted by GitBox <gi...@apache.org>.
pan3793 closed pull request #3336: Use StageAttempt instead of StageId in SQLOperationListener
URL: https://github.com/apache/incubator-kyuubi/pull/3336


-- 
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: notifications-unsubscribe@kyuubi.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


[GitHub] [incubator-kyuubi] cxzl25 commented on pull request #3336: Use StageAttempt instead of StageId in SQLOperationListener

Posted by GitBox <gi...@apache.org>.
cxzl25 commented on PR #3336:
URL: https://github.com/apache/incubator-kyuubi/pull/3336#issuecomment-1227982964

   ```java
   ERROR AsyncEventQueue: Listener SQLOperationListener threw an exception
   java.lang.NullPointerException
           at org.apache.spark.kyuubi.SQLOperationListener.onTaskEnd(SQLOperationListener.scala:155)
           at org.apache.spark.scheduler.SparkListenerBus.doPostEvent(SparkListenerBus.scala:45)
           at org.apache.spark.scheduler.SparkListenerBus.doPostEvent$(SparkListenerBus.scala:28)
           at org.apache.spark.scheduler.AsyncEventQueue.doPostEvent(AsyncEventQueue.scala:37)
           at org.apache.spark.scheduler.AsyncEventQueue.doPostEvent(AsyncEventQueue.scala:37)
           at org.apache.spark.util.ListenerBus.postToAll(ListenerBus.scala:117)
           at org.apache.spark.util.ListenerBus.postToAll$(ListenerBus.scala:101)
           at org.apache.spark.scheduler.AsyncEventQueue.super$postToAll(AsyncEventQueue.scala:105)
           at org.apache.spark.scheduler.AsyncEventQueue.$anonfun$dispatch$1(AsyncEventQueue.scala:105)
           at scala.runtime.java8.JFunction0$mcJ$sp.apply(JFunction0$mcJ$sp.java:23)
           at scala.util.DynamicVariable.withValue(DynamicVariable.scala:62)
           at org.apache.spark.scheduler.AsyncEventQueue.org$apache$spark$scheduler$AsyncEventQueue$$dispatch(AsyncEventQueue.scala:100)
           at org.apache.spark.scheduler.AsyncEventQueue$$anon$2.$anonfun$run$1(AsyncEventQueue.scala:96)
           at org.apache.spark.util.Utils$.tryOrStopSparkContext(Utils.scala:1442)
           at org.apache.spark.scheduler.AsyncEventQueue$$anon$2.run(AsyncEventQueue.scala:96)
   ```
   
   


-- 
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: notifications-unsubscribe@kyuubi.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org