You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ze...@apache.org on 2022/06/13 13:50:14 UTC

[arrow] branch master updated: ARROW-16813: [Go][Parquet] fix go parquet dictionary encoding writer property

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

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


The following commit(s) were added to refs/heads/master by this push:
     new c5a74802b8 ARROW-16813: [Go][Parquet] fix go parquet dictionary encoding writer property
c5a74802b8 is described below

commit c5a74802b8e02ba882ff09393b8e1afd9d28270f
Author: Matt DePero <de...@neeva.co>
AuthorDate: Mon Jun 13 09:50:03 2022 -0400

    ARROW-16813: [Go][Parquet] fix go parquet dictionary encoding writer property
    
    Closes #13357 from mdepero/parquetgo-dictprop
    
    Authored-by: Matt DePero <de...@neeva.co>
    Signed-off-by: Matthew Topol <mt...@factset.com>
---
 go/parquet/writer_properties.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/go/parquet/writer_properties.go b/go/parquet/writer_properties.go
index 0363b67ca0..123cf212eb 100644
--- a/go/parquet/writer_properties.go
+++ b/go/parquet/writer_properties.go
@@ -109,7 +109,7 @@ func WithDictionaryDefault(dict bool) WriterProperty {
 // WithDictionaryFor allows enabling or disabling dictionary encoding for a given column path string
 func WithDictionaryFor(path string, dict bool) WriterProperty {
 	return func(cfg *writerPropConfig) {
-		cfg.dictEnabled[path] = true
+		cfg.dictEnabled[path] = dict
 	}
 }