[GLUTEN-3711][VL] Fix printConf to Support Velox Config #3710

Co-authored-by: Ted Jenks <[email protected]>
apache · Nov 15, 2023 · 36c9208 · 36c9208
1 parent 857872e
commit 36c9208
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/cpp/velox/compute/VeloxBackend.cc b/cpp/velox/compute/VeloxBackend.cc
@@ -108,10 +108,10 @@ const std::string kVeloxShuffleReaderPrintFlag = "spark.gluten.velox.shuffleRead
 
 namespace gluten {
 
-void VeloxBackend::printConf(const std::unordered_map<std::string, std::string>& conf) {
+void VeloxBackend::printConf(const facebook::velox::Config& conf) {
   std::ostringstream oss;
   oss << "STARTUP: VeloxBackend conf = {\n";
-  for (auto& [k, v] : conf) {
+  for (auto& [k, v] : conf.values()) {
     oss << " {" << k << ", " << v << "}\n";
   }
   oss << "}\n";

diff --git a/cpp/velox/compute/VeloxBackend.h b/cpp/velox/compute/VeloxBackend.h
@@ -63,7 +63,7 @@ class VeloxBackend {
 
   void initJolFilesystem(const facebook::velox::Config* conf);
 
-  void printConf(const std::unordered_map<std::string, std::string>& conf);
+  void printConf(const facebook::velox::Config& conf);
 
   std::string getCacheFilePrefix() {
     return "cache." + boost::lexical_cast<std::string>(boost::uuids::random_generator()()) + ".";