Skip to content
This repository has been archived by the owner on Sep 20, 2022. It is now read-only.

[HIVEMALL-149] Add tiny script for updating resources/ddl/define-* #120

Closed
wants to merge 3 commits into from

Conversation

takuti
Copy link
Member

@takuti takuti commented Oct 4, 2017

What changes were proposed in this pull request?

Add a script for updating resources/ddl/define-*

What type of PR is it?

Improvement

What is the Jira issue?

https://issues.apache.org/jira/browse/HIVEMALL-149

How to use this feature?

./bin/add_define.sh

@coveralls
Copy link

coveralls commented Oct 4, 2017

Coverage Status

Coverage remained the same at 41.109% when pulling 01005d7 on takuti:HIVEMALL-149-define into 7bb5d04 on apache:master.

Copy link
Member

@myui myui left a comment

Choose a reason for hiding this comment

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

update_ddls.sh is more straightforward script name.

@takuti
Copy link
Member Author

takuti commented Oct 4, 2017

@myui updated with README

Copy link
Member

@myui myui left a comment

Choose a reason for hiding this comment

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

@takuti LGTM. Please merge this PR with squashing.

@coveralls
Copy link

coveralls commented Oct 4, 2017

Coverage Status

Coverage increased (+0.003%) to 41.112% when pulling a84ba87 on takuti:HIVEMALL-149-define into 7bb5d04 on apache:master.

@coveralls
Copy link

coveralls commented Oct 4, 2017

Coverage Status

Coverage remained the same at 41.109% when pulling a84ba87 on takuti:HIVEMALL-149-define into 7bb5d04 on apache:master.

@asfgit asfgit closed this in d4f4ab9 Oct 4, 2017
@takuti takuti deleted the HIVEMALL-149-define branch October 4, 2017 09:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants