GitHub user HuangWHWHW opened a pull request:

    https://github.com/apache/flink/pull/1131

    [hotfix][Table API tests]add toDataSet in table API tests

    See the discussion in the PR:https://github.com/apache/flink/pull/1098 for 
detail.
    And I'm not sure this change need a JIRA or just take a hotfix.
    
    The problem is due to the ambiguous Table APIs between DataSet.scala and 
DataStream.scala.
    The class Table can call methods in both DataSet.scala and DataStream.scala.
    The class Table need to specify `toDataSet` or `toDataStream` firstly when 
there are the same methods in both DataSet.scala and DataStream.scala.
    So I added the `toDataSet`(since there is only one method in DataSet.scala 
recently but further will be another in DataStream.scala) in these Table API 
tests for the follow-up working in DataStream.scala.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/HuangWHWHW/flink FLINK-2622-toDataSet

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/1131.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1131
    
----
commit 62741ca7cfe6620d94717fe9eb6909e34da763d0
Author: HuangWHWHW <[email protected]>
Date:   2015-09-15T06:28:17Z

    [hotfix][Table API tests]add toDataSet in table API tests

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to