Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

midje-cascalog produces wrong output from basic queries #292

Open
timothypratley opened this issue Sep 30, 2015 · 7 comments
Open

midje-cascalog produces wrong output from basic queries #292

timothypratley opened this issue Sep 30, 2015 · 7 comments

Comments

@timothypratley
Copy link
Contributor

using [cascalog/midje-cascalog "2.1.1"]

(fact
 (<- [?n]
     (num-pair :> ?n ?n))
 => (produces [[0] [1] [4]]]))
;; false, produces [[4] [4] [4]]

(?<- (stdout) [?n]
     (num-pair :> ?n ?n))
;; prints 0 1 4

Also, the tests for midje-cascalog do not run:

✘-1 ~/git/cascalog/midje-cascalog [develop|✔]
17:03 $ lein midje

-> Exception

@timothypratley timothypratley changed the title produces wrong output from basic queries midje-cascalog produces wrong output from basic queries Sep 30, 2015
@sritchie
Copy link
Collaborator

sritchie commented Oct 1, 2015

Well, doesn't the first clause say that it produces a single 3-tuple?


Sent from Mailbox

On Wed, Sep 30, 2015 at 5:47 PM, Timothy Pratley notifications@github.com
wrote:

using [cascalog/midje-cascalog "2.1.1"]

(fact
 (<- [?n]
     (num-pair :> ?n ?n))
 => (produces [[0 1 4]]))
;; false, produces [[4] [4] [4]]
(?<- (stdout) [?n]
     (num-pair :> ?n ?n))
;; prints 0 1 4

Reply to this email directly or view it on GitHub:
#292

@timothypratley
Copy link
Contributor Author

@sritchie I am expecting [[0] [1] [4]] but get [[4] [4] [4]] instead... does that clarify?
num-pair is from cascalog.playground, there are pairs of 0s 1s and 4s in the num-pairs data.

@sritchie
Copy link
Collaborator

sritchie commented Oct 1, 2015

I think The library wraps the tests in a config that has io.sort.mb equal to 1. Wonder if that's what's going on.


Sent from Mailbox

On Wed, Sep 30, 2015 at 7:43 PM, Timothy Pratley notifications@github.com
wrote:

@sritchie I am expecting [[0] [1] [4]] but get [[4] [4] [4]] instead... does that clarify?

num-pair is from cascalog.playground, there are pairs of 0s 1s and 4s in the num-pairs data.

Reply to this email directly or view it on GitHub:
#292 (comment)

@danielstjules
Copy link

Just wondering if there's been any update on this? Is there a last known good version we could jump to? I've tried a few with no luck.

@danielstjules
Copy link

Turns out it's only an issue with [org.clojure/clojure "1.7.0"]. Things work fine with [org.clojure/clojure "1.6.0"] :)

@sritchie
Copy link
Collaborator

sritchie commented Nov 6, 2015

Nice, this is good to know. I don't have any time to work on this, but happy to help guide if you guys want take a stab at it and have any questions about the code.

@jiyouyou125
Copy link
Contributor

@sritchie add a test, test success. it seems related iterator-seq. #298 add Implicit equality constraints test,related #292

sritchie added a commit that referenced this issue Jan 12, 2016
add Implicit equality constraints test,related #292
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

No branches or pull requests

4 participants