🐛 Source Greenhouse: faster check stream

[etiennecallies]

What

Fixes #45411 + a minor typo in connector's README.md.

How

In the issue, I mentioned skip_count parameter, but it would require to create a new stream, that would be used only for check. I preferred to change the stream check from applications to users, as there are far less users than applications. In a company there are indeed more applicants than recruiters.

User Impact

Normally, there always are some users, therefore check should always pass. By the way, it is more likely to have some users and no applications, than no users and some applications.

Can this PR be safely reverted and rolled back?

• YES 💚
• NO ❌

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
airbyte-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 17, 2024 5:06pm

[etiennecallies]

Before the fix:

(airbyte) 15:53:23 ~/airbyte/airbyte-integrations/connectors/source-greenhouse (source-greenhouse *) $ time poetry run source-greenhouse check --config secrets/config.json
{"type": "LOG", "log": {"level": "INFO", "message": "Check succeeded"}}
{"type": "CONNECTION_STATUS", "connectionStatus": {"status": "SUCCEEDED"}}

real	0m37.324s
user	0m0.948s
sys	0m0.154s

After the fix:

(airbyte) 15:56:24 ~/airbyte/airbyte-integrations/connectors/source-greenhouse (source-greenhouse *) $ time poetry run source-greenhouse check --config secrets/config.json
{"type": "LOG", "log": {"level": "INFO", "message": "Check succeeded"}}
{"type": "CONNECTION_STATUS", "connectionStatus": {"status": "SUCCEEDED"}}

real	0m2.167s
user	0m0.964s
sys	0m0.150s

[marcosmarxm]

Nice catch @etiennecallies ! LGTM

[marcosmarxm]

Running CI tests