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