sql
sql copied to clipboard
Split off doctests from main tests
Description
Following in the footsteps of #3070, this PR splits doctests into its own step too. The general motivation behind both changes is that both suites can start failing due to factors outside this repo (doctests if CLI changes, BWC if lucene codec changes), but integ tests should always pass. Since issues were recently caused by missing failing integ tests (hidden by doctest/BWC failures that were unrelated to the changes), having them in their own checks to make integ test failures visible at a glance would have prevented the issue. It also makes the actions run faster.
Related Issues
N/A
Check List
- [ ] New functionality includes testing.
- [ ] New functionality has been documented.
- [ ] New functionality has javadoc added.
- [ ] New functionality has a user manual doc added.
- [ ] API changes companion pull request created.
- [ ] Commits are signed per the DCO using
--signoff
. - [ ] Public documentation issue/PR created.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.