(spark) branch master updated: [SPARK-48171][CORE] Clean up the use of deprecated constructors of `o.rocksdb.Logger`

[dongjoon]

This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new c326f3c143ff [SPARK-48171][CORE] Clean up the use of deprecated 
constructors of `o.rocksdb.Logger`
c326f3c143ff is described below

commit c326f3c143ffdd56954706aeb4e0b82ac819bf03
Author: yangjie01 <yangji...@baidu.com>
AuthorDate: Tue May 7 07:33:38 2024 -0700

    [SPARK-48171][CORE] Clean up the use of deprecated constructors of 
`o.rocksdb.Logger`
    
    ### What changes were proposed in this pull request?
    This pr aims to clean up the use of deprecated constructors of 
`o.rocksdb.Logger`, the change ref to
    
    
https://github.com/facebook/rocksdb/blob/5c2be544f5509465957706c955b6d623e889ac4e/java/src/main/java/org/rocksdb/Logger.java#L39-L54
    
    ```
    /**
       * <p>AbstractLogger constructor.</p>
       *
       * <p><strong>Important:</strong> the log level set within
       * the {link org.rocksdb.Options} instance will be used as
       * maximum log level of RocksDB.</p>
       *
       * param options {link org.rocksdb.Options} instance.
       *
       * deprecated Use {link Logger#Logger(InfoLogLevel)} instead, e.g. {code 
new
       *     Logger(options.infoLogLevel())}.
       */
      Deprecated
      public Logger(final Options options) {
        this(options.infoLogLevel());
      }
    ```
    
    ### Why are the changes needed?
    Clean up deprecated api usage.
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    Pass GitHub Actions
    
    ### Was this patch authored or co-authored using generative AI tooling?
    No
    
    Closes #46436 from LuciferYang/rocksdb-deprecation.
    
    Authored-by: yangjie01 <yangji...@baidu.com>
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
 .../src/main/java/org/apache/spark/network/util/RocksDBProvider.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/common/network-common/src/main/java/org/apache/spark/network/util/RocksDBProvider.java
 
b/common/network-common/src/main/java/org/apache/spark/network/util/RocksDBProvider.java
index f3b7b48355a0..2b5ea01d94c9 100644
--- 
a/common/network-common/src/main/java/org/apache/spark/network/util/RocksDBProvider.java
+++ 
b/common/network-common/src/main/java/org/apache/spark/network/util/RocksDBProvider.java
@@ -136,7 +136,7 @@ public class RocksDBProvider {
         private static final Logger LOG = 
LoggerFactory.getLogger(RocksDBLogger.class);
 
         RocksDBLogger(Options options) {
-          super(options);
+          super(options.infoLogLevel());
         }
 
         @Override


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org