fix: Support param timeout when persisting

[sudohainguyen]

What this PR does / why we need it: to support declaring timeout when persisting historical retrieval job

Which issue(s) this PR fixes:

Fixes #

[feast-ci-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: sudohainguyen
To complete the pull request process, please assign felixwang9817 after the PR has been reviewed.
You can assign the PR to them by writing /assign @felixwang9817 in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[gbmarc1]

Hello @sudohainguyen !
I am working on a similar PR but for to_df and to_arrow methods.

• I think it would be good to add unit-tests to those methods.
• What about Optional[int] with default None if you don't want a timeout.

[sudohainguyen]

Hello @sudohainguyen ! I am working on a similar PR but for to_df and to_arrow methods.

• I think it would be good to add unit-tests to those methods.

sure let me see

[sudohainguyen]

• What about Optional[int] with default None if you don't want a timeout.

@gbmarc1 there should always be a timeout I believe

[sudohainguyen]

just rebased

[sudohainguyen]

@achals do you think we still need this PR?