Skip to content

Comments

Gradle tweaks#10

Merged
snazy merged 3 commits intoapache:mainfrom
snazy:gradle-adoptions
Aug 5, 2024
Merged

Gradle tweaks#10
snazy merged 3 commits intoapache:mainfrom
snazy:gradle-adoptions

Conversation

@snazy
Copy link
Member

@snazy snazy commented Jul 30, 2024

  • configure-on-demand
  • replace usages of deprecated Gradle APIs

@snazy snazy force-pushed the gradle-adoptions branch from 294d8fd to a3bdad6 Compare July 31, 2024 18:57
@snazy snazy marked this pull request as ready for review July 31, 2024 18:57
@snazy snazy force-pushed the gradle-adoptions branch from a3bdad6 to 250cd06 Compare August 1, 2024 12:43
build.gradle Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

then shouldn't we just change the version? I'm confused why we would have this workaround?

@RussellSpitzer
Copy link
Member

I really think we should break this one into separate PR's. The posfix -> . refactor I think is good but we are also doing a bunch of other things here.

@snazy snazy requested a review from a team as a code owner August 5, 2024 07:24
snazy added 3 commits August 5, 2024 09:24
* configure-on-demand
* replace usages of deprecated Gradle APIs
@snazy snazy force-pushed the gradle-adoptions branch 2 times, most recently from 85bb9d1 to 2f4151b Compare August 5, 2024 07:28
@snazy
Copy link
Member Author

snazy commented Aug 5, 2024

I really think we should break this one into separate PR's. The posfix -> . refactor I think is good but we are also doing a bunch of other things here.

That's all for the "replace deprecated Gradle API" part (buildDir is deprecated).
sourceCompatibility/targetCompatibiltiy didn't really work, see #76 (usage of newer Java APIs).

}

project(":polaris-service") {
apply plugin: "application"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For my own reference, Plugin specification is moved into project specific build files.

@snazy snazy merged commit 1f6655b into apache:main Aug 5, 2024
@snazy snazy deleted the gradle-adoptions branch August 5, 2024 09:12
eric-maynard pushed a commit to eric-maynard/polaris that referenced this pull request Dec 12, 2024
…rop-table

Add foreign table support for listTables and dropTable
eric-maynard pushed a commit to eric-maynard/polaris that referenced this pull request Feb 15, 2025
snazy added a commit that referenced this pull request Jan 15, 2026
The "Spark Client Regression Tests" CI job requires some disk space to operate. With just a little bit of added "content", the job will fail to `no space left on device` during the `docker compose` invocation building an image. Such errors make it impossible to get the log from the workflow, unless you capture the log before the workflow runs into the `no space left on device` situation. With "no space left", GitHub workflow infra is unable to capture the logs.

```
 #10 ERROR: failed to copy files: userspace copy failed: write /home/spark/polaris/v3.5/integration/build/2.13/quarkus-build/gen/quarkus-app/lib/main/com.google.http-client.google-http-client-1.47.1.jar: no space left on device
```

This change is a stop-gap solution to prevent this error from happening for now.
evindj pushed a commit to evindj/polaris that referenced this pull request Jan 26, 2026
…#3429)

The "Spark Client Regression Tests" CI job requires some disk space to operate. With just a little bit of added "content", the job will fail to `no space left on device` during the `docker compose` invocation building an image. Such errors make it impossible to get the log from the workflow, unless you capture the log before the workflow runs into the `no space left on device` situation. With "no space left", GitHub workflow infra is unable to capture the logs.

```
 apache#10 ERROR: failed to copy files: userspace copy failed: write /home/spark/polaris/v3.5/integration/build/2.13/quarkus-build/gen/quarkus-app/lib/main/com.google.http-client.google-http-client-1.47.1.jar: no space left on device
```

This change is a stop-gap solution to prevent this error from happening for now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants