-
Notifications
You must be signed in to change notification settings - Fork 5
V0.1.1 : pytest for BigQueryToCSV.extract() #83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
Thu Oct 24 9:25 PM IST
… guiding users to resolution ETAs eventually
…e repository for resolution
…nical specifications.
and config.metadata["source"]["engine"] == SupportedEngine.BIGQUERY.value | ||
): | ||
data_loader = BigQueryToCSV(config=config) | ||
data_loader.load() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Todo: Write test cases
Please Add Test case for the following functionalities :
|
session_catalog=catalog, | ||
identifier = identifier | ||
) | ||
stats = table_statistics_from_iceberg_metadata( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#1 I am trying to solve for the statistics here @rakhioza07 @redpheonixx
from local_data_platform.store.dimension import Dimension | ||
from local_data_platform.product.vocab import Vocab | ||
|
||
def test_get(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love how simple this is 🥇
topic: pyiceberg subject: table.scan().count() to-do: implement table.scan().count() eta: Nov 28
Please review this test and the code changes along with it
name: Pull request
about: Implement an solution for this project
title: ''
labels: ''
assignees: ''
Is your pull request related to a issue? Please tag.
Describe the solution you have implemented
Describe why alternatives you considered are not optimal
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the pull request here.