Parquet: add a default value of COMPRESSION_LEVEL=-1 to please QGIS (master only)

Author: rouault

ogr/ogrsf_frmts/parquet/ogr_parquet.h

@@ -24,6 +24,8 @@
 #include "../arrow_common/ogr_arrow.h"
 #include "ogr_include_parquet.h"
 
+constexpr int DEFAULT_COMPRESSION_LEVEL = -1;
+
 /************************************************************************/
 /*                       OGRParquetLayerBase                            */
 /************************************************************************/

ogr/ogrsf_frmts/parquet/ogrparquetdriver.cpp

@@ -728,12 +728,16 @@ void OGRParquetDriver::InitMetadata()
         auto psOption = CPLCreateXMLNode(oTree.get(), CXT_Element, "Option");
         CPLAddXMLAttributeAndValue(psOption, "name", "COMPRESSION_LEVEL");
         CPLAddXMLAttributeAndValue(psOption, "type", "int");
-        CPLAddXMLAttributeAndValue(psOption, "min",
-                                   CPLSPrintf("%d", minComprLevel));
+        CPLAddXMLAttributeAndValue(
+            psOption, "min",
+            CPLSPrintf("%d",
+                       std::min(DEFAULT_COMPRESSION_LEVEL, minComprLevel)));
         CPLAddXMLAttributeAndValue(psOption, "max",
                                    CPLSPrintf("%d", maxComprLevel));
         CPLAddXMLAttributeAndValue(psOption, "description",
                                    osCompressionLevelDesc.c_str());
+        CPLAddXMLAttributeAndValue(psOption, "default",
+                                   CPLSPrintf("%d", DEFAULT_COMPRESSION_LEVEL));
     }
 
     {

ogr/ogrsf_frmts/parquet/ogrparquetwriterlayer.cpp

@@ -456,7 +456,11 @@ bool OGRParquetWriterLayer::SetOptions(CSLConstList papszOptions,
     const char *pszCompressionLevel =
         CSLFetchNameValue(papszOptions, "COMPRESSION_LEVEL");
     if (pszCompressionLevel)
-        m_oWriterPropertiesBuilder.compression_level(atoi(pszCompressionLevel));
+    {
+        const int nCompressionLevel = atoi(pszCompressionLevel);
+        if (nCompressionLevel != DEFAULT_COMPRESSION_LEVEL)
+            m_oWriterPropertiesBuilder.compression_level(nCompressionLevel);
+    }
 
     const std::string osCreator =
         CSLFetchNameValueDef(papszOptions, "CREATOR", "");