bvaradar commented on code in PR #10554:
URL: https://github.com/apache/hudi/pull/10554#discussion_r1463747359
##########
hudi-spark-datasource/hudi-spark/src/test/scala/org/apache/spark/sql/hudi/TestInsertTable.scala:
##########
@@ -2160,6 +2172,7 @@ class TestInsertTable extends HoodieSparkSqlTestBase {
|union
|select '1' as id, 'aa' as name, 123 as dt, '2023-10-12' as `day`,
12 as `hour`
|""".stripMargin)
+ spark.sparkContext.addSparkListener(new
StageParallelismListener(stageName = "collect at
HoodieSparkEngineContext.java"))
Review Comment:
@xuzifu666 : Relying of class name string here would be highly fragile here.
Can you make this explicit dependent to the class and also validate if the
callback is indeed being called.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
