neilconway opened a new pull request, #26577:
URL: https://github.com/apache/datafusion/pull/26577

   ## Which issue does this PR close?
   
   - Related to #25088, #25148
   
   ## Rationale for this change
   
   Many PRs modify some Rust code and also touch some SLT files. At present, 
this results in checking all of `datafusion-sqllogictest` for semver 
compatibility, which is very expensive (because it depends on almost all of the 
other crates). `datafusion-sqllogictest` is a published crate but it is test 
tooling that isn't part of the library API, so I think skipping semver checks 
for it is not unreasonable.
   
   The longer-term goal here is to make the semver check cheap enough that we 
can run it using `ubuntu-slim`, but we're probably not at that point yet. In 
any case, reducing semver check overhead is a useful improvement by itself.
   
   ## What changes are included in this PR?
   
   See above. We also use `install-action` to install `protoc` rather than 
`apt`, since it's much faster.
   
   ## What is the testing strategy for this PR?
   
   CI change only.
   
   ## Are there any user-facing changes?
   
   No.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to