Skip to content

Conversation

@rchromik
Copy link
Contributor

@rchromik rchromik commented Jan 10, 2024

Feature or Bugfix

  • Bugfix

Detail

  • Fix for _read_parquet_chunked function if columns are specified as parameter.
  • By submitting this PR, only fields will be passed to the schema when reading data from chunks

Relates

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@rchromik rchromik changed the title bugfix: Fix reading of chunked parquet when columns parameter is spec… fix: Fix reading of chunked parquet when columns parameter is spec… Jan 10, 2024
@malachi-constant
Copy link
Contributor

AWS CodeBuild CI Report

  • CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
  • Commit ID: 1ddc58c
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@malachi-constant
Copy link
Contributor

AWS CodeBuild CI Report

  • CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
  • Commit ID: 39f3bd2
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@rchromik
Copy link
Contributor Author

rchromik commented Jan 10, 2024

I think the build issue is due to pull request #2594 which was already merged with a failed checks.

@malachi-constant
Copy link
Contributor

AWS CodeBuild CI Report

  • CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
  • Commit ID: 39f3bd2
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@rchromik rchromik marked this pull request as draft January 10, 2024 17:41
@rchromik rchromik marked this pull request as ready for review January 10, 2024 17:41
@rchromik rchromik closed this Jan 10, 2024
@rchromik rchromik deleted the bugfix/chunked_read_parquet_fix_columns_param branch January 10, 2024 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants