[CI][Dev][Swift] Is ci/scripts/swift_test.sh still necessary?

[hiroyuki-sato]

Describe the enhancement requested

#46803 Removed swift implementation.

ci/scripts/swift_test.sh still exists and docker-compose.yml use this file.

arrow/docker-compose.yml

Lines 1042 to 1054 in f082d31

	ubuntu-swift:
	# Usage:
	# docker compose build ubuntu-swift
	# docker compose run --rm ubuntu-swift
	# Parameters:
	image: ubuntu-swift
	build:
	context: .
	dockerfile: ci/docker/ubuntu-swift.dockerfile
	shm_size: *shm-size
	volumes: *ubuntu-volumes
	command: >
	/bin/bash -c "/arrow/ci/scripts/swift_test.sh /arrow"

Is this part still necessary?

It may have planned to delete it in another PR, but I just wanted to double check.

Component(s)

Continuous Integration

[raulcd]

sorry, those should have been removed! Good catch!

[hiroyuki-sato]

Hello, @raulcd. Thank you for your comment. As a reporter, I created #46807. If you create a PR by yourself, I'll close it.

It seems that the following part needs update about Swift part. But I can't find README.md in arrow-swift repository. So I didn't change in #46807.

arrow/docs/source/implementations.rst

Lines 73 to 75 in f082d31

	* - Swift
	- `Swift Docs <https://github.com/apache/arrow/blob/main/swift/Arrow/README.md>`_
	- `Swift Source <https://github.com/apache/arrow/tree/main/swift>`_

[raulcd]

We already are pointing docs to to https://arrow.apache.org/swift/ as seen here:
https://github.com/apache/arrow-swift/blob/36e45ef04a48108ee7593f5925c36f19d0df2541/.asf.yaml#L22

This does not exist yet but it is supposed to exist once we release swift in the future. I think we can just update with that URL, @kou what do you think? or should we move this README to the top level swift-repo? https://github.com/apache/arrow-swift/blob/36e45ef04a48108ee7593f5925c36f19d0df2541/Arrow/README.md

BTW there is an issue to create the README:

• Add a basic README file arrow-swift#23

As per the Swift Source we can update to https://github.com/apache/arrow-swift

[hiroyuki-sato]

FYI: Implementations used above URL.

And It seems that this link also need update. I couldn't find the source.

https://arrow.apache.org

[assignUser]

Issue resolved by pull request 46807
#46807

[hiroyuki-sato]

Thanks. JFYI: apache/arrow-site#660 created

[kou]

This does not exist yet but it is supposed to exist once we release swift in the future. I think we can just update with that URL, @kou what do you think?

Let's use this approach! (We use https://arrow.apache.org/swift/ that doesn't exist yet but will be available soon.)