[GitHub] metamodel pull request: METAMODEL-218: Fixed

2015-12-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metamodel/pull/76 --- 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 en

[GitHub] metamodel pull request: METAMODEL-218: Fixed

2015-12-09 Thread drexler42
Github user drexler42 commented on the pull request: https://github.com/apache/metamodel/pull/76#issuecomment-163246456 Let's merge then! --- 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

[GitHub] metamodel pull request: METAMODEL-218: Fixed

2015-12-09 Thread kaspersorensen
Github user kaspersorensen commented on the pull request: https://github.com/apache/metamodel/pull/76#issuecomment-163219986 I merged in master ... Was just a conflict in CHANGES.md. But let's see that Travis builds correctly before we merge to master --- If your project is set up fo

[GitHub] metamodel pull request: METAMODEL-218: Fixed

2015-12-09 Thread drexler42
Github user drexler42 commented on the pull request: https://github.com/apache/metamodel/pull/76#issuecomment-163211807 I can copy the customers table directly into Postgres now. The fix works. A word of warning though: - MetaModel 4.4.2-SNAPSHOT is not compatible with DataCleaner

[GitHub] metamodel pull request: METAMODEL-218: Fixed

2015-12-09 Thread drexler42
Github user drexler42 commented on the pull request: https://github.com/apache/metamodel/pull/76#issuecomment-163144698 Not yet, I'm afraid. Having trouble building a DataCleaner with this MM in it. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] metamodel pull request: METAMODEL-218: Fixed

2015-12-08 Thread kaspersorensen
Github user kaspersorensen commented on the pull request: https://github.com/apache/metamodel/pull/76#issuecomment-163020477 Did you come to any conclusion @drexler42 ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If y

[GitHub] metamodel pull request: METAMODEL-218: Fixed

2015-12-08 Thread drexler42
Github user drexler42 commented on the pull request: https://github.com/apache/metamodel/pull/76#issuecomment-162811307 I'll give it a go and provide feedback --- 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 projec

[GitHub] metamodel pull request: METAMODEL-218: Fixed

2015-12-07 Thread kaspersorensen
GitHub user kaspersorensen opened a pull request: https://github.com/apache/metamodel/pull/76 METAMODEL-218: Fixed This fixes METAMODEL-218 by adding a few return statement in DefaultQueryRewriter. Also updated the test template to ensure that it's working on all the datab