[BEAM-4536] Remove with_attributes keyword from ReadFromPubSub.

[udim]

Cherrypick for 2.5.0

---

Follow this checklist to help us incorporate your contribution quickly and easily:

• Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
• If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

[udim]

R: @jbonofre

[jbonofre]

Same issue on this PR: java.lang.IllegalStateException: sendHeaders has already been called.

[boyuanzz]

retest this please

[udim]

Looking at java breakage.

[boyuanzz]

Run Java PreCommit

[udim]

retest this please

[boyuanzz]

Run Java PreCommit

[udim]

@jbonofre ./gradlew :beam-runners-direct-java:test was broken for me on my local machine. I did a git pull --rebase and that fixed it. I then pushed the commit again in the hopes that it'd fix the precommit somehow, however now #5611 has disabled running Java precommits.
I've asked @swegner to revert #5611 so we can verify that Java precommits are not broken.

[udim]

Run Java PreCommit

[pabloem]

I triggered a build on the release branch itself, and it failed:
https://builds.apache.org/job/beam_PreCommit_Java_GradleBuild/6441/consoleFull

[swegner]

FYI, the failing test org.apache.beam.runners.direct.portable.ReferenceRunnerTest.pipelineExecution, I recognize from other PRs as being flaky (history). This PR seems unrelated and seems unlikely that it caused the regression.

/cc @reuvenlax @tgroh as the owners for this test.