Skip to content

Conversation

@iemejia
Copy link
Member

@iemejia iemejia commented Sep 24, 2019

I took #674 and fixed the issues that broke, let's see if CI is happy this time.

@nandorKollar
Copy link

@iemejia I'm afraid that Hive mostly cares about patches attached to Jiras - at least the How to contribute tells about patches and doesn't mentione PRs. I think that for a PR, not all of the tests that are relevant are executed (I guess mvn clean install -DskipTests -q -Pitests from travis.yaml isn't too much), but I'm not sure that.

@iemejia
Copy link
Member Author

iemejia commented Sep 25, 2019

Oh I see thanks @nandorKollar I will move this there then.

@iemejia
Copy link
Member Author

iemejia commented Sep 26, 2019

I am closing this one in favor of the attached patch in HIVE-21737

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