Skip to content

Commit

Permalink
Remove bigquery prefix
Browse files Browse the repository at this point in the history
  • Loading branch information
zhilingc committed Apr 22, 2020
1 parent fb302ac commit 2832d80
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions serving/src/main/resources/application.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,9 @@ feast:
# Please omit the trailing slash in the URI.
staging_location: gs://mybucket/myprefix
# Retry options for BigQuery retrieval jobs
bigquery_initial_retry_delay_secs: 1
initial_retry_delay_seconds: 1
# BigQuery timeout for retrieval jobs
bigquery_total_timeout_secs: 21600
total_timeout_seconds: 21600
subscriptions:
- name: "*"
project: "*"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ public static HistoricalRetriever create(Map<String, String> config) {
BigQuery bigquery = BigQueryOptions.getDefaultInstance().getService();
Storage storage = StorageOptions.getDefaultInstance().getService();

String jobStagingLocation = config.get("staging-location");
String jobStagingLocation = config.get("staging_location");
if (!jobStagingLocation.contains("://")) {
throw new IllegalArgumentException(
String.format("jobStagingLocation is not a valid URI: %s", jobStagingLocation));
Expand All @@ -74,8 +74,8 @@ public static HistoricalRetriever create(Map<String, String> config) {
.setDatasetId(config.get("dataset_id"))
.setProjectId(config.get("project_id"))
.setJobStagingLocation(config.get("staging_location"))
.setInitialRetryDelaySecs(Integer.parseInt(config.get("bigquery_initial_retry_delay_secs")))
.setTotalTimeoutSecs(Integer.parseInt(config.get("bigquery_total_timeout_secs")))
.setInitialRetryDelaySecs(Integer.parseInt(config.get("initial_retry_delay_seconds")))
.setTotalTimeoutSecs(Integer.parseInt(config.get("total_timeout_seconds")))
.setStorage(storage)
.build();
}
Expand Down

0 comments on commit 2832d80

Please sign in to comment.