Skip to content

ARROW-9844: [CI] Add Go build job on s390x#8047

Closed
vivkong wants to merge 1 commit into
apache:masterfrom
linux-on-ibm-z:master-travis-s390x
Closed

ARROW-9844: [CI] Add Go build job on s390x#8047
vivkong wants to merge 1 commit into
apache:masterfrom
linux-on-ibm-z:master-travis-s390x

Conversation

@vivkong

@vivkong vivkong commented Aug 25, 2020

Copy link
Copy Markdown
Contributor

As suggested by @kou in #8011, this will add a Travis CI job for Go on s390x.

@vivkong vivkong force-pushed the master-travis-s390x branch from bc8b13d to 706c73e Compare August 25, 2020 12:29
@github-actions

Copy link
Copy Markdown

Comment thread .travis.yml Outdated
@vivkong vivkong force-pushed the master-travis-s390x branch from 706c73e to 130e12f Compare August 26, 2020 12:36

@kou kou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@kou kou closed this in 92e01cc Aug 26, 2020
@vivkong

vivkong commented Aug 26, 2020

Copy link
Copy Markdown
Contributor Author

@kou Sorry I'm not familiar with the process, will this PR be merged?

@kou

kou commented Aug 26, 2020

Copy link
Copy Markdown
Member

This PR has been merged into master: 92e01cc

We use our merge tool https://github.com/apache/arrow/blob/master/dev/merge_arrow_pr.py instead of using GitHub's merge UI to update corresponding JIRA issue too. We close PR but the change is merged.

Now, you can use the CI job in #8011 by rebasing on master.

@emkornfield

Copy link
Copy Markdown
Contributor

We should probably hold off merging more PRs for big endian architectures until we come to consensus on this on the ML about support in general

@kou

kou commented Aug 27, 2020

Copy link
Copy Markdown
Member

@vivkong

vivkong commented Aug 27, 2020

Copy link
Copy Markdown
Contributor Author

Right I understand. Thanks @emkornfield and @kou.

@vivkong vivkong deleted the master-travis-s390x branch August 27, 2020 17:39
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.

3 participants