srowen commented on a change in pull request #26467: [SPARK-29477]Improve tooltip for Streaming tab URL: https://github.com/apache/spark/pull/26467#discussion_r352780059
########## File path: streaming/src/test/scala/org/apache/spark/streaming/UISeleniumSuite.scala ########## @@ -162,7 +163,7 @@ class UISeleniumSuite outputOpIds.map(_.text) should be (List("0", "1")) // Check job ids - val jobIdCells = findAll(cssSelector( """#batch-job-table a""")).toSeq + val jobIdCells = findAll(cssSelector( """#jobId""")).toSeq Review comment: As I say, I am not sure it makes sense to modify the user facing HTML just for this. Just ignore the "?" here in the test. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org