justinmclean opened a new issue, #7207:
URL: https://github.com/apache/gravitino/issues/7207

   ### What would you like to be improved?
   
   > Task :core:javadoc
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/tag/TagDispatcher.java:36:
 warning: no @param for metalake
     String[] listTags(String metalake);
              ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/tag/TagDispatcher.java:43:
 warning: no @param for metalake
     Tag[] listTagsInfo(String metalake);
           ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/listener/api/info/partitions/PartitionInfo.java:75:
 warning: no @param for partition
     public static PartitionInfo of(Partition partition) {
                                 ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/RelationalBackend.java:61:
 warning: no @param for <E>
     <E extends Entity & HasIdentifier> List<E> list(
                                                ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/RelationalBackend.java:83:
 warning: no @param for <E>
     <E extends Entity & HasIdentifier> void insert(E e, boolean overwritten)
                                             ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/RelationalBackend.java:95:
 warning: no @param for <E>
     <E extends Entity & HasIdentifier> E update(
                                          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/RelationalBackend.java:95:
 warning: no @param for updater
     <E extends Entity & HasIdentifier> E update(
                                          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/RelationalBackend.java:108:
 warning: no @param for <E>
     <E extends Entity & HasIdentifier> E get(NameIdentifier ident, 
Entity.EntityType entityType)
                                          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/SupportsRelationOperations.java:51:
 warning: no @param for <E>
     default <E extends Entity & HasIdentifier> List<E> listEntitiesByRelation(
                                                        ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/SupportsRelationOperations.java:69:
 warning: no @param for <E>
     <E extends Entity & HasIdentifier> List<E> listEntitiesByRelation(
                                                ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/JDBCDatabase.java:30:
 warning: no description for @throws
      * @throws RuntimeException
        ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/session/SqlSessions.java:110:
 warning: no @param for <T>
     public static <T> T getMapper(Class<T> className) {
                         ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/authorization/AccessControlDispatcher.java:296:
 warning: no @param for metalake
     Role grantPrivilegeToRole(
          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/authorization/AccessControlDispatcher.java:311:
 warning: no @param for metalake
     Role revokePrivilegesFromRole(
          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/converters/SQLExceptionConverter.java:34:
 warning: no @throws for java.io.IOException
     void toGravitinoException(SQLException sqlException, Entity.EntityType 
type, String name)
          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/catalog/PartitionDispatcher.java:56:
 warning: no description for @throws
      * @throws NoSuchPartitionException
        ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/utils/POConverters.java:188:
 warning: no @param for metalakeId
     public static CatalogPO initializeCatalogPOWithVersion(
                             ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/utils/POConverters.java:217:
 warning: no @param for metalakeId
     public static CatalogPO updateCatalogPOWithVersion(
                             ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/utils/POConverters.java:287:
 warning: no @param for builder
     public static SchemaPO initializeSchemaPOWithVersion(
                            ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/utils/POConverters.java:376:
 warning: no @param for builder
     public static TablePO initializeTablePOWithVersion(
                           ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/utils/POConverters.java:544:
 warning: no @param for builder
     public static FilesetPO initializeFilesetPOWithVersion(
                             ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/utils/POConverters.java:587:
 warning: no @param for needUpdateVersion
     public static FilesetPO updateFilesetPOWithVersion(
                             ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/utils/POConverters.java:777:
 warning: no @param for builder
     public static UserPO initializeUserPOWithVersion(UserEntity userEntity, 
UserPO.Builder builder) {
                          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/utils/POConverters.java:1023:
 warning: no @param for builder
     public static RolePO initializeRolePOWithVersion(RoleEntity roleEntity, 
RolePO.Builder builder) {
                          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/storage/relational/utils/POConverters.java:1045:
 warning: no @param for builder
     public static GroupPO initializeGroupPOWithVersion(
                           ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/meta/CatalogEntity.java:137:
 warning: no @return
     public CatalogInfo toCatalogInfo() {
                        ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/connector/authorization/BaseAuthorization.java:55:
 warning: no @param for metalake
     public abstract AuthorizationPlugin newPlugin(
                                         ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/connector/authorization/BaseAuthorization.java:55:
 warning: no @param for catalogProvider
     public abstract AuthorizationPlugin newPlugin(
                                         ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/connector/authorization/BaseAuthorization.java:55:
 warning: no @param for config
     public abstract AuthorizationPlugin newPlugin(
                                         ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/listener/api/event/GetPartitionFailureEvent.java:42:
 warning: no @param for partitionName
     public GetPartitionFailureEvent(
            ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/listener/api/event/LinkModelVersionFailureEvent.java:41:
 warning: no @param for linkModelVersionRequest
     public LinkModelVersionFailureEvent(
            ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/listener/api/event/ListModelVersionsEvent.java:35:
 warning: no @param for versions
     public ListModelVersionsEvent(String user, NameIdentifier identifier, 
int[] versions) {
            ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/listener/api/event/PartitionExistsFailureEvent.java:42:
 warning: no @param for partitionName
     public PartitionExistsFailureEvent(
            ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/listener/api/event/RegisterAndLinkModelEvent.java:63:
 warning: no description for @return
      * @return
        ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/connector/PropertiesMetadata.java:98:
 warning: no @param for propertyName
     default boolean containsProperty(String propertyName) {
                     ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/connector/CatalogOperations.java:62:
 warning: no @throws for java.lang.Exception
     void testConnection(
          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/listener/api/info/Either.java:92:
 warning: no @return
     public static <L, R> Either<L, R> left(L value) {
                                       ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/listener/api/info/Either.java:104:
 warning: no @return
     public static <L, R> Either<L, R> right(R value) {
                                       ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/audit/AuditLogWriter.java:38:
 warning: no description for @param
      * @param properties
        ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/audit/AuditLogWriter.java:40:
 warning: no @param for formatter
     void init(Formatter formatter, Map<String, String> properties);
          ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/audit/AuditLogWriter.java:45:
 warning: no description for @param
      * @param auditLog
        ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/audit/AuditLogWriter.java:52:
 warning: no description for @param
      * @param event
        ^
   
/Users/justin/gravitino/core/src/main/java/org/apache/gravitino/audit/AuditLogWriter.java:62:
 warning: no description for @return
      * @return
        ^
   43 warnings
   
   ### How should we improve?
   
   Fix Javadoc issues


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