Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: Fix minor things on communication doc #1006

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kavirajk
Copy link

Which issue does this PR close?

Closes NA.

Rationale for this change

Having correct links on communication page is generally helpful and particularly for new contributors

What changes are included in this PR?

Changes:

  1. Good first issue link was broken & was pointing wrongly to datafusion repository
  2. Update repo links (datafusion -> ballista)
  3. Remove spec reference that was pointing to datafusion specification.

Are there any user-facing changes?

NO

NO

Changes:
1. `Good first issue` link was broken & was pointing to `datafusion` repository
2. Update repo links (`datafusion` -> `ballista`)
3. Remove spec reference that was pointing to `datafusion`.

Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com>
Copy link
Member

@andygrove andygrove left a comment

Choose a reason for hiding this comment

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

Thanks @kavirajk

@kavirajk
Copy link
Author

@andygrove any idea CI (Docker stage) is not passing. Looks totally unrelated to the PR.

[3/9] COPY target/release/ballista-scheduler /root/ballista-scheduler
#6 ERROR: failed to calculate checksum of ref abc794c5-012e-49ab-b658-c7ad62c56cb6::00d8a19ciim46e09f5gy4g04f: "/target/release/ballista-scheduler": not found

#7 [5/9] RUN chmod a+x /root/ballista-scheduler &&     chmod a+x /root/ballista-executor
#7 CACHED

#8 [6/9] RUN mkdir -p /data &&     wget -q https://d37ci6vzurychx.cloudfront.net/trip-data/yellow_tripdata_2022-01.parquet -P /data/
#8 CACHED

#9 [7/9] COPY ballista/scheduler/ui/build /var/www/html
#9 ERROR: failed to calculate checksum of ref abc794c5-012e-49ab-b658-c7ad62c56cb6::00d8a19ciim[46](https://github.com/apache/datafusion-ballista/actions/runs/8775461142/job/24159808966?pr=1006#step:5:47)e09f5gy4g04f: "/ballista/scheduler/ui/build": not found

#10 [4/9] COPY target/release/ballista-executor /root/ballista-executor
#10 ERROR: failed to calculate checksum of ref abc794c5-012e-49ab-b658-c7ad62c56cb6::00d8a19ciim46e09f5gy4g04f: "/target/release/ballista-executor": not found

Looks like copying anything from /target/release/ and ui/build is failing.

Also this may be the one blocking other PRs as well?

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.

None yet

2 participants