Re: [PR] PARQUET-2483: Update references to parquet-mr in Readmes [parquet-java]

2024-05-25 Thread via GitHub
emkornfield commented on code in PR #1358: URL: https://github.com/apache/parquet-java/pull/1358#discussion_r1615013171 ## README.md: ## @@ -17,7 +17,7 @@ ~ under the License. --> -Parquet MR [![Build Status](https://github.com/apache/parquet-mr/workflows/Test/badge.svg

Re: [PR] Add: `sum_value` to statistics and ColumnIndex [parquet-format]

2024-05-25 Thread via GitHub
emkornfield closed pull request #83: Add: `sum_value` to statistics and ColumnIndex URL: https://github.com/apache/parquet-format/pull/83 -- 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

Re: [PR] Add: `sum_value` to statistics and ColumnIndex [parquet-format]

2024-05-25 Thread via GitHub
emkornfield commented on PR #83: URL: https://github.com/apache/parquet-format/pull/83#issuecomment-2132045826 Given this is six years old at this point. going to remove this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[PR] Bump org.codehaus.mojo:exec-maven-plugin from 3.2.0 to 3.3.0 [parquet-java]

2024-05-25 Thread via GitHub
dependabot[bot] opened a new pull request, #1359: URL: https://github.com/apache/parquet-java/pull/1359 Bumps [org.codehaus.mojo:exec-maven-plugin](https://github.com/mojohaus/exec-maven-plugin) from 3.2.0 to 3.3.0. Release notes Sourced from https://github.com/mojohaus/exec-maven

Re: [PR] DRAFT: Parquet 3 metadata with decoupled column metadata [parquet-format]

2024-05-25 Thread via GitHub
emkornfield commented on code in PR #242: URL: https://github.com/apache/parquet-format/pull/242#discussion_r1614611250 ## src/main/thrift/parquet.thrift: ## @@ -1165,6 +1317,62 @@ struct FileMetaData { 9: optional binary footer_signing_key_metadata } +/** Metadata for a c

Re: [PR] DRAFT: Parquet 3 metadata with decoupled column metadata [parquet-format]

2024-05-25 Thread via GitHub
emkornfield commented on code in PR #242: URL: https://github.com/apache/parquet-format/pull/242#discussion_r1614611250 ## src/main/thrift/parquet.thrift: ## @@ -1165,6 +1317,62 @@ struct FileMetaData { 9: optional binary footer_signing_key_metadata } +/** Metadata for a c

Re: [PR] DRAFT: Parquet 3 metadata with decoupled column metadata [parquet-format]

2024-05-25 Thread via GitHub
mapleFU commented on code in PR #242: URL: https://github.com/apache/parquet-format/pull/242#discussion_r1614502880 ## src/main/thrift/parquet.thrift: ## Review Comment: > I thought only max amd min fields are deprecated. Sigh, a unrelated issue is that currently `m

Re: [PR] DRAFT: Parquet 3 metadata with decoupled column metadata [parquet-format]

2024-05-25 Thread via GitHub
pitrou commented on code in PR #242: URL: https://github.com/apache/parquet-format/pull/242#discussion_r1614489120 ## src/main/thrift/parquet.thrift: ## @@ -1165,6 +1317,62 @@ struct FileMetaData { 9: optional binary footer_signing_key_metadata } +/** Metadata for a column

Re: [PR] PARQUET-2483: Update references to parquet-mr in Readmes [parquet-java]

2024-05-25 Thread via GitHub
wgtmac commented on code in PR #1358: URL: https://github.com/apache/parquet-java/pull/1358#discussion_r1614402839 ## README.md: ## @@ -17,7 +17,7 @@ ~ under the License. --> -Parquet MR [![Build Status](https://github.com/apache/parquet-mr/workflows/Test/badge.svg)](ht