Skip to content

ARROW-7117: [C++][CI] Fix the hanging C++ tests in Windows 2019#5915

Closed
pitrou wants to merge 1 commit intoapache:masterfrom
pitrou:ARROW-7117-gh-actions-windows
Closed

ARROW-7117: [C++][CI] Fix the hanging C++ tests in Windows 2019#5915
pitrou wants to merge 1 commit intoapache:masterfrom
pitrou:ARROW-7117-gh-actions-windows

Conversation

@pitrou
Copy link
Copy Markdown
Member

@pitrou pitrou commented Nov 27, 2019

The fix is simply to use normal shared linking.

The fix is simply to use normal shared linking.
ARROW_VERBOSE_THIRDPARTY_BUILD: OFF
steps:
- name: Disable Crash Dialogs
run: reg add "HKCU\SOFTWARE\Microsoft\Windows\Windows Error Reporting" /v DontShowUI /t REG_DWORD /d 1 /f
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@kszucs Do you want me to put this in a script?

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.

This is quite environment specific so this is the right place.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This can actually be useful on all Windows builders.

Copy link
Copy Markdown
Member Author

@pitrou pitrou Nov 27, 2019

Choose a reason for hiding this comment

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

Note this is not required for this PR, but was part of my investigations and can save future hassles.

Copy link
Copy Markdown
Member

@kszucs kszucs Nov 27, 2019

Choose a reason for hiding this comment

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

@pitrou then what was the solution? Just the linking or the hanging was just transient?

@github-actions
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@kszucs kszucs left a comment

Choose a reason for hiding this comment

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

LGTM

@kszucs
Copy link
Copy Markdown
Member

kszucs commented Nov 27, 2019

@pitrou Currently appveyor is the bottleneck for our overall CI times.

What do you think about removing https://github.com/apache/arrow/blob/master/appveyor.yml#L65-L68 ?

@pitrou
Copy link
Copy Markdown
Member Author

pitrou commented Nov 27, 2019

I'd rather move it to GH Actions :-)

@pitrou
Copy link
Copy Markdown
Member Author

pitrou commented Nov 27, 2019

@kszucs
Copy link
Copy Markdown
Member

kszucs commented Nov 27, 2019

My question is about how can we reach feature parity with that build.

@pitrou
Copy link
Copy Markdown
Member Author

pitrou commented Nov 27, 2019

Ah. Well, regardless of feature parity, there are different dimensions being tested:

  • different version of the MSVC compiler
  • different CMake generators (msbuild vs Ninja)

So even if we enable features on the Win2019 build, we will still benefit from another build with Ninja and an older MSVC.

@kszucs
Copy link
Copy Markdown
Member

kszucs commented Nov 27, 2019

Which MSVC version do we use on appveyor?

@kszucs
Copy link
Copy Markdown
Member

kszucs commented Nov 27, 2019

The appveyor error is unrelated and probably transient, merging.

@kszucs kszucs closed this in 9f40178 Nov 27, 2019
@pitrou
Copy link
Copy Markdown
Member Author

pitrou commented Nov 27, 2019

Visual Studio 2015 on Windows Server 2012 is the oldest version, AFAIK.
Feel free to open a JIRA for creating new GH workflows. Also, remember that we're also testing Python on AppVeyor.

@pitrou pitrou deleted the ARROW-7117-gh-actions-windows branch November 27, 2019 14:16
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