Skip to content

Commit

Permalink
Removed time.sleep() after init_online_lookup() in online feature sto…
Browse files Browse the repository at this point in the history
…re samples. (#2647)
  • Loading branch information
ynpandey authored Sep 12, 2023
1 parent 2f3d0b8 commit 875c20b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -776,7 +776,7 @@
"cell_type": "markdown",
"metadata": {},
"source": [
"The following code cell tracks completion of the backfill job. Using the premium tier Azure Cache for Redis provisioned earlier, this step may take approximately 3-4 minutes to complete."
"The following code cell tracks completion of the backfill job. Using the premium tier Azure Cache for Redis provisioned earlier, this step may take approximately 5 minutes to complete."
]
},
{
Expand All @@ -801,7 +801,7 @@
"source": [
"# Get the job URL, and stream the job logs.\n",
"# With PREMIUM Redis SKU, SKU family \"P\", and cache capacity 2,\n",
"# it takes approximately 3-4 minutes to complete.\n",
"# it takes approximately 5 minutes to complete.\n",
"fs_client.jobs.stream(poller.result().job_id)"
]
},
Expand Down Expand Up @@ -882,10 +882,7 @@
"import time\n",
"\n",
"# Initialize the online store client.\n",
"init_online_lookup(features, AzureMLOnBehalfOfCredential())\n",
"\n",
"# It may take 5 - 10 sec the local RPC server of online store client to be ready.\n",
"time.sleep(5)"
"init_online_lookup(features, AzureMLOnBehalfOfCredential())"
]
},
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,6 @@ def init():

init_online_lookup(features, credential)

time.sleep(20)

logging.info("Init complete")


Expand Down

0 comments on commit 875c20b

Please sign in to comment.