LuciferYang commented on code in PR #50999:
URL: https://github.com/apache/spark/pull/50999#discussion_r2104975035


##########
tools/src/main/scala/org/apache/spark/tools/GenerateMIMAIgnore.scala:
##########
@@ -157,7 +157,7 @@ object GenerateMIMAIgnore {
    * and subpackages both from directories and jars present on the classpath.
    */
   private def getClasses(packageName: String): Set[String] = {
-    val finder = ClassFinder(maybeOverrideAsmVersion = Some(Opcodes.ASM7))
+    val finder = ClassFinder(maybeOverrideAsmVersion = Some(Opcodes.ASM8))

Review Comment:
   for fix 
   
   ```
   Exception in thread "main" java.lang.UnsupportedOperationException: Record 
requires ASM8
        at 
org.objectweb.asm.ClassVisitor.visitRecordComponent(ClassVisitor.java:323)
        at 
org.objectweb.asm.ClassReader.readRecordComponent(ClassReader.java:954)
        at org.objectweb.asm.ClassReader.accept(ClassReader.java:732)
        at org.objectweb.asm.ClassReader.accept(ClassReader.java:425)
        at org.clapper.classutil.asm.ClassFile$.load(ClassFinderImpl.scala:224)
        at org.clapper.classutil.ClassFinder.classData(ClassFinder.scala:404)
        at 
org.clapper.classutil.ClassFinder.$anonfun$processOpenZip$2(ClassFinder.scala:359)
        at scala.collection.Iterator$$anon$9.next(Iterator.scala:584)
        at scala.collection.Iterator$$anon$10.nextCur(Iterator.scala:594)
        at scala.collection.Iterator$$anon$10.hasNext(Iterator.scala:608)
        at 
scala.collection.immutable.LazyList$.scala$collection$immutable$LazyList$$stateFromIterator(LazyList.scala:1195)
        at 
scala.collection.immutable.LazyList$.$anonfun$stateFromIterator$1(LazyList.scala:1195)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state$lzycompute(LazyList.scala:282)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state(LazyList.scala:273)
        at scala.collection.immutable.LazyList.isEmpty(LazyList.scala:292)
        at 
scala.collection.immutable.LazyList.$anonfun$lazyAppendedAll$1(LazyList.scala:401)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state$lzycompute(LazyList.scala:282)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state(LazyList.scala:273)
        at scala.collection.immutable.LazyList.isEmpty(LazyList.scala:292)
        at 
scala.collection.immutable.LazyList.$anonfun$lazyAppendedAll$1(LazyList.scala:401)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state$lzycompute(LazyList.scala:282)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state(LazyList.scala:273)
        at scala.collection.immutable.LazyList.isEmpty(LazyList.scala:292)
        at 
scala.collection.immutable.LazyList.$anonfun$mapImpl$1(LazyList.scala:539)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state$lzycompute(LazyList.scala:282)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state(LazyList.scala:273)
        at scala.collection.immutable.LazyList.isEmpty(LazyList.scala:292)
        at 
scala.collection.immutable.LazyList$.$anonfun$filterImpl$1(LazyList.scala:1036)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state$lzycompute(LazyList.scala:282)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state(LazyList.scala:273)
        at scala.collection.immutable.LazyList.isEmpty(LazyList.scala:292)
        at 
scala.collection.immutable.LazyList$.$anonfun$filterImpl$1(LazyList.scala:1036)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state$lzycompute(LazyList.scala:282)
        at 
scala.collection.immutable.LazyList.scala$collection$immutable$LazyList$$state(LazyList.scala:273)
        at scala.collection.immutable.LazyList.isEmpty(LazyList.scala:292)
        at 
scala.collection.immutable.LazyList$LazyIterator.hasNext(LazyList.scala:1274)
        at scala.collection.mutable.Growable.addAll(Growable.scala:61)
        at scala.collection.mutable.Growable.addAll$(Growable.scala:57)
        at scala.collection.immutable.SetBuilderImpl.addAll(Set.scala:405)
        at scala.collection.immutable.Set$.from(Set.scala:362)
        at scala.collection.IterableOnceOps.toSet(IterableOnce.scala:1469)
        at scala.collection.IterableOnceOps.toSet$(IterableOnce.scala:1469)
        at scala.collection.AbstractIterable.toSet(Iterable.scala:935)
        at 
org.apache.spark.tools.GenerateMIMAIgnore$.getClasses(GenerateMIMAIgnore.scala:165)
        at 
org.apache.spark.tools.GenerateMIMAIgnore$.privateWithin(GenerateMIMAIgnore.scala:64)
        at 
org.apache.spark.tools.GenerateMIMAIgnore$.main(GenerateMIMAIgnore.scala:131)
        at 
org.apache.spark.tools.GenerateMIMAIgnore.main(GenerateMIMAIgnore.scala)
   
   ```



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