Skip to content

Provide descriptive error on invalid table reference #1627

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.

Sign up for GitHub

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

Merged
feast-ci-bot merged 20 commits into from
Jun 21, 2021
Merged

Provide descriptive error on invalid table reference #1627

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
1a91a33
Initial commit to catch nonexistent table
Jun 7, 2021
da7a712
simplify nonexistent BQ table test
Jun 8, 2021
f6a2f98
clean up table_exists exception
Jun 8, 2021
41e69a7
remove unneeded variable
Jun 8, 2021
af937c4
function name change to _assert_table_exists
Jun 8, 2021
215de4b
Initial commit to catch nonexistent table
Jun 7, 2021
caac9d0
simplify nonexistent BQ table test
Jun 8, 2021
2d2a484
clean up table_exists exception
Jun 8, 2021
c548ab6
function name change to _assert_table_exists
Jun 8, 2021
d5eb8af
fix lint errors and rebase
Jun 9, 2021
21a7631
Fix get_table(None) error
Jun 9, 2021
50831ef
custom exception for both missing file and BQ source
Jun 9, 2021
4e7a803
revert FileSource checks
Jun 10, 2021
2d99b05
Use DataSourceNotFoundException instead of subclassing
Jun 11, 2021
c513660
Moved assert_table_exists out of the BQ constructor to apply_total
Jun 14, 2021
617dde1
rename test and test asset
Jun 16, 2021
6b3975a
move validate logic back to data_source
Jun 17, 2021
458beab
fixed tests
Jun 19, 2021
5abc428
Set pytest.integration for tests that access BQ
Jun 20, 2021
4ec510d
Import pytest in failed test files
Jun 20, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
function name change to _assert_table_exists 
Signed-off-by: Cody Lin <codyl@twitter.com>
  • Loading branch information
Cody Lin committed Jun 19, 2021
commit c548ab6481fc432c52cf85c61de6d19fbc23564d
11 changes: 0 additions & 11 deletions sdk/python/feast/data_source.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -635,19 +635,8 @@ def __init__(
query: Optional[str] = None,
):
self._bigquery_options = BigQueryOptions(table_ref=table_ref, query=query)
<<<<<<< HEAD
self._assert_table_exists()

if not self._table_exists():
raise NameError(
f"""
Unable to find table {table_ref} in BigQuery. Please check that table exists.
"""
)
=======
self._table_exists()
>>>>>>> c6935e99... clean up table_exists exception

super().__init__(
event_timestamp_column,
created_timestamp_column,
Expand Down