Skip to content

[FLINK-5189] Delete Row and its related classes from table api#3004

Closed
tonycox wants to merge 7 commits into
apache:masterfrom
tonycox:FLINK-5189
Closed

[FLINK-5189] Delete Row and its related classes from table api#3004
tonycox wants to merge 7 commits into
apache:masterfrom
tonycox:FLINK-5189

Conversation

@tonycox
Copy link
Copy Markdown
Contributor

@tonycox tonycox commented Dec 14, 2016

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

  • General

    • The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
    • The pull request addresses only one issue
    • Each commit in the PR has a meaningful commit message (including the JIRA id)
  • Documentation

    • Documentation has been added for new functionality
    • Old documentation affected by the pull request has been updated
    • JavaDoc for public methods has been added
  • Tests & Build

    • Functionality added by the pull request is covered by tests
    • mvn clean verify has been executed successfully locally or a Travis build has passed

@tonycox tonycox changed the title [Flink-5189] Delete Row and its related classes from table api [FLINK-5189] Delete Row and its related classes from table api Dec 14, 2016
@fhueske
Copy link
Copy Markdown
Contributor

fhueske commented Dec 14, 2016

Thanks @tonycox.
PR looks good.

@fhueske
Copy link
Copy Markdown
Contributor

fhueske commented Dec 15, 2016

Merging

@asfgit asfgit closed this in a9e6ec8 Dec 15, 2016
joseprupi pushed a commit to joseprupi/flink that referenced this pull request Feb 12, 2017
hequn8128 pushed a commit to hequn8128/flink that referenced this pull request Jun 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants