Add BasePrimitiveJob class and deprecate PrimitiveJob.submit method

[t-imamichi]

Summary

Add BasePrimitiveJob class and deprecate PrimitiveJob.submit method

Resolves #11541

TODO

• reno

Details and comments

[qiskit-bot]

One or more of the the following people are requested to review this:

• @Qiskit/terra-core
• @ajavadia
• @ikkoham
• @levbishop
• @t-imamichi

[coveralls]

Pull Request Test Coverage Report for Build 7537622138

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.01%) to 89.352%

---

Totals
Change from base Build 7508541254:	0.01%
Covered Lines:	59540
Relevant Lines:	66635

---

💛 - Coveralls

[jyu00]

I think the other JobV1 methods need to also go into BasePrimitiveJob.

Essentially BasePrimitiveJob would look exactly like JobV1 except

• no more submit()
• no more backend()
• __init__ doesn't take backend
• status() returns a generic Status and result() returns a generic Result()

[ihincks]

Since Status is now generic, I guess it makes sense to expose done(), cancelled(), running(), etc as abstract methods so that this information can be had without knowing too much about the status object, which is what @jyu00 is also suggesting. I do wonder if we should deprecate wait_for_final_state() in addition to submit() since it seems a bit like bloat---I usually see people just calling result() when they want to block.

[t-imamichi]

I copied all JobV1 methods except submit and wait_for_final_state.

[ihincks]

I left some relatively minor comments. Other than those, this LGTM.

Also, I should clarify that I'm not strongly against keeping wait_for_final_state, it just seems like an obvious (to me) place to reduce clutter.

[t-imamichi]

I agree with your idea to deprecate wait_for_final_state. If there is no objection, how about deprecating it?

[ihincks]

@jyu00 may still want to review.

Edit: @jyu00 said I can go ahead with merge.