turboFei commented on code in PR #2644:
URL: https://github.com/apache/celeborn/pull/2644#discussion_r1690909405


##########
project/CelebornBuild.scala:
##########
@@ -446,10 +446,12 @@ object Utils {
 }
 
 object CelebornSpi {
-  lazy val spi = Project("celeborn-spi", file("spi")).
-    settings(
+  lazy val spi = Project("celeborn-spi", file("spi"))
+    .settings(
       commonSettings,
-      releaseSettings
+      releaseSettings,
+      crossPaths := false,
+      Compile / doc / javacOptions := Seq("-encoding", UTF_8.name(), 
"-source", "1.8")

Review Comment:
   Fix doc issue:
   Before:
   ```
   build/sbt "clean;celeborn-spi/doc"
   Using /Users/fwang12/.sdkman/candidates/java/current as default JAVA_HOME.
   Note, this will be overridden by -java-home if it is set.
   [info] welcome to sbt 1.9.4 (Azul Systems, Inc. Java 1.8.0_392)
   [info] loading settings for project celeborn-build from plugins.sbt ...
   [info] loading project definition from /Users/fwang12/todo/celeborn/project
   [info] compiling 1 Scala source to 
/Users/fwang12/todo/celeborn/project/target/scala-2.12/sbt-1.0/classes ...
   [info] loading settings for project celeborn from version.sbt ...
   [info] resolving key references (12502 settings) ...
   [info] set current project to celeborn (in build 
file:/Users/fwang12/todo/celeborn/)
   [success] Total time: 0 s, completed Jul 24, 2024 11:43:09 PM
   [info] Main Java API documentation to 
/Users/fwang12/todo/celeborn/spi/target/api...
   Provided by Standard doclet:
     -d <directory>                   Destination directory for output files
     -use                             Create class and package usage pages
     -version                         Include @version paragraphs
     -author                          Include @author paragraphs
     -docfilessubdirs                 Recursively copy doc-file subdirectories
     -splitindex                      Split index into one file per letter
     -windowtitle <text>              Browser window title for the documentation
     -doctitle <html-code>            Include title for the overview page
     -header <html-code>              Include header text for each page
     -footer <html-code>              Include footer text for each page
     -top    <html-code>              Include top text for each page
     -bottom <html-code>              Include bottom text for each page
     -link <url>                      Create links to javadoc output at <url>
     -linkoffline <url> <url2>        Link to docs at <url> using package list 
at <url2>
     -excludedocfilessubdir <name1>:.. Exclude any doc-files subdirectories 
with given name.
     -group <name> <p1>:<p2>..        Group specified packages together in 
overview page
     -nocomment                       Suppress description and tags, generate 
only declarations.
     -nodeprecated                    Do not include @deprecated information
     -noqualifier <name1>:<name2>:... Exclude the list of qualifiers from the 
output.
     -nosince                         Do not include @since information
     -notimestamp                     Do not include hidden time stamp
     -nodeprecatedlist                Do not generate deprecated list
     -notree                          Do not generate class hierarchy
     -noindex                         Do not generate index
     -nohelp                          Do not generate help link
     -nonavbar                        Do not generate navigation bar
     -serialwarn                      Generate warning about @serial tag
     -tag <name>:<locations>:<header> Specify single argument custom tags
     -taglet                          The fully qualified name of Taglet to 
register
     -tagletpath                      The path to Taglets
     -charset <charset>               Charset for cross-platform viewing of 
generated documentation.
     -helpfile <file>                 Include file that help link links to
     -linksource                      Generate source in HTML
     -sourcetab <tab length>          Specify the number of spaces each tab 
takes up in the source
     -keywords                        Include HTML meta tags with package, 
class and member info
     -stylesheetfile <path>           File to change style of the generated 
documentation
     -docencoding <name>              Specify the character encoding for the 
output
   [warn] Unexpected javac output: javadoc: error - invalid flag: -g.
   [info] Usage: javadoc [options] [packagenames] [sourcefiles] [@files]
   [info]   -overview <file>                 Read overview documentation from 
HTML file
   [info]   -public                          Show only public classes and 
members
   [info]   -protected                       Show protected/public classes and 
members (default)
   [info]   -package                         Show package/protected/public 
classes and members
   [info]   -private                         Show all classes and members
   [info]   -help                            Display command line options and 
exit
   [info]   -doclet <class>                  Generate output via alternate 
doclet
   [info]   -docletpath <path>               Specify where to find doclet class 
files
   [info]   -sourcepath <pathlist>           Specify where to find source files
   [info]   -classpath <pathlist>            Specify where to find user class 
files
   [info]   -cp <pathlist>                   Specify where to find user class 
files
   [info]   -exclude <pkglist>               Specify a list of packages to 
exclude
   [info]   -subpackages <subpkglist>        Specify subpackages to recursively 
load
   [info]   -breakiterator                   Compute first sentence with 
BreakIterator
   [info]   -bootclasspath <pathlist>        Override location of class files 
loaded
   [info]                                    by the bootstrap class loader
   [info]   -source <release>                Provide source compatibility with 
specified release
   [info]   -extdirs <dirlist>               Override location of installed 
extensions
   [info]   -verbose                         Output messages about what Javadoc 
is doing
   [info]   -locale <name>                   Locale to be used, e.g. en_US or 
en_US_WIN
   [info]   -encoding <name>                 Source file encoding name
   [info]   -quiet                           Do not display status messages
   [info]   -J<flag>                         Pass <flag> directly to the 
runtime system
   [info]   -X                               Print a synopsis of nonstandard 
options and exit
   [info] 1 error
   [warn] javadoc exited with exit code 1
   [error] sbt.inc.Doc$JavadocGenerationFailed
   [error]         at 
sbt.inc.Doc$.sbt$inc$Doc$$$anonfun$cachedJavadoc$1(Doc.scala:51)
   [error]         at sbt.inc.Doc$$anonfun$cachedJavadoc$2.run(Doc.scala:41)
   [error]         at sbt.inc.Doc$.sbt$inc$Doc$$$anonfun$prepare$1(Doc.scala:62)
   [error]         at sbt.inc.Doc$$anonfun$prepare$5.run(Doc.scala:57)
   [error]         at sbt.inc.Doc$.go$1(Doc.scala:73)
   [error]         at sbt.inc.Doc$.$anonfun$cached$5(Doc.scala:82)
   [error]         at sbt.inc.Doc$.$anonfun$cached$5$adapted(Doc.scala:81)
   [error]         at 
sbt.util.Tracked$.$anonfun$inputChangedW$1(Tracked.scala:220)
   [error]         at sbt.inc.Doc$.sbt$inc$Doc$$$anonfun$cached$1(Doc.scala:85)
   [error]         at sbt.inc.Doc$$anonfun$cached$7.run(Doc.scala:68)
   [error]         at 
sbt.Defaults$.$anonfun$docTaskSettings$4(Defaults.scala:2178)
   [error]         at scala.Function1.$anonfun$compose$1(Function1.scala:49)
   [error]         at 
sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:63)
   [error]         at sbt.std.Transform$$anon$4.work(Transform.scala:69)
   [error]         at sbt.Execute.$anonfun$submit$2(Execute.scala:283)
   [error]         at 
sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:24)
   [error]         at sbt.Execute.work(Execute.scala:292)
   [error]         at sbt.Execute.$anonfun$submit$1(Execute.scala:283)
   [error]         at 
sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
   [error]         at 
sbt.CompletionService$$anon$2.call(CompletionService.scala:65)
   [error]         at java.util.concurrent.FutureTask.run(FutureTask.java:266)
   [error]         at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
   [error]         at java.util.concurrent.FutureTask.run(FutureTask.java:266)
   [error]         at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
   [error]         at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
   [error]         at java.lang.Thread.run(Thread.java:750)
   [error] (celeborn-spi / Compile / doc) sbt.inc.Doc$JavadocGenerationFailed
   ```
   
   After:
   ```
   build/sbt "clean;celeborn-spi/doc"
   
   Using /Users/fwang12/.sdkman/candidates/java/current as default JAVA_HOME.
   Note, this will be overridden by -java-home if it is set.
   [info] welcome to sbt 1.9.4 (Azul Systems, Inc. Java 1.8.0_392)
   [info] loading settings for project celeborn-build from plugins.sbt ...
   [info] loading project definition from /Users/fwang12/todo/celeborn/project
   [info] loading settings for project celeborn from version.sbt ...
   [info] resolving key references (12502 settings) ...
   [info] set current project to celeborn (in build 
file:/Users/fwang12/todo/celeborn/)
   [success] Total time: 0 s, completed Jul 24, 2024 11:53:43 PM
   [info] Main Java API documentation to 
/Users/fwang12/todo/celeborn/spi/target/api...
   [info] Loading source file 
/Users/fwang12/todo/celeborn/spi/src/main/java/org/apache/celeborn/spi/authentication/PasswordCredential.java...
   [info] Loading source file 
/Users/fwang12/todo/celeborn/spi/src/main/java/org/apache/celeborn/spi/authentication/TokenCredential.java...
   [info] Loading source file 
/Users/fwang12/todo/celeborn/spi/src/main/java/org/apache/celeborn/spi/authentication/Credential.java...
   [info] Loading source file 
/Users/fwang12/todo/celeborn/spi/src/main/java/org/apache/celeborn/spi/authentication/PasswdAuthenticationProvider.java...
   [info] Loading source file 
/Users/fwang12/todo/celeborn/spi/src/main/java/org/apache/celeborn/spi/authentication/TokenAuthenticationProvider.java...
   [info] Constructing Javadoc information...
   [info] Standard Doclet version 1.8.0_392
   [info] Building tree for all the packages and classes...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/org/apache/celeborn/spi/authentication/Credential.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/org/apache/celeborn/spi/authentication/PasswdAuthenticationProvider.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/org/apache/celeborn/spi/authentication/PasswordCredential.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/org/apache/celeborn/spi/authentication/TokenAuthenticationProvider.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/org/apache/celeborn/spi/authentication/TokenCredential.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/org/apache/celeborn/spi/authentication/package-frame.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/org/apache/celeborn/spi/authentication/package-summary.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/org/apache/celeborn/spi/authentication/package-tree.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/constant-values.html...
   [info] Building index for all the packages and classes...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/overview-tree.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/index-all.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/deprecated-list.html...
   [info] Building index for all classes...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/allclasses-frame.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/allclasses-noframe.html...
   [info] Generating /Users/fwang12/todo/celeborn/spi/target/api/index.html...
   [info] Generating 
/Users/fwang12/todo/celeborn/spi/target/api/help-doc.html...
   [info] Main Java API documentation successful.
   [success] Total time: 1 s, completed Jul 24, 2024 11:53:44 PM
   
   ```



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

Reply via email to