NICTA / scoobi

A Scala productivity framework for Hadoop.
http://nicta.github.com/scoobi/
482 stars 97 forks source link

QuickStart example doesn't compile #284

Closed fancellu closed 11 years ago

fancellu commented 11 years ago

persist(counts.toTextFile(args(1))

needs an extra ) at the end

Once that is done I find another compile error.

type mismatch; [error] found : (Int, Int) => Int [error] required: com.nicta.scoobi.core.Reduction[?] [error] .combine((a: Int, b: Int) => a + b) [error] ^

At the end it points to WordCount in the examples, which is also a 404 error.

Its not a good start for a "quick start"

espringe commented 11 years ago

There were some API changes which broke compile (so it'll need to be updated), but in the mean time before it and the link is fixed, here's the actual word count example which should compile:

https://github.com/NICTA/scoobi/tree/master/examples/wordCount

On Mon, Jul 15, 2013 at 4:48 PM, Dino Fancellu notifications@github.comwrote:

persist(counts.toTextFile(args(1))

needs an extra ) at the end

Once that is done I find another compile error.

type mismatch; [error] found : (Int, Int) => Int [error] required: com.nicta.scoobi.core.Reduction[?] [error] .combine((a: Int, b: Int) => a + b) [error] ^

At the end it points to WordCount in the examples, which is also a 404 error.

Its not a good start for a "quick start"

— Reply to this email directly or view it on GitHubhttps://github.com/NICTA/scoobi/issues/284 .

espringe commented 11 years ago

The API change btw is: .combine((a: Int, b: Int) => a + b)

is now

.combine(Sum.int)

On Mon, Jul 15, 2013 at 4:57 PM, Eric Springer ericwspringer@gmail.com wrote:

There were some API changes which broke compile (so it'll need to be updated), but in the mean time before it and the link is fixed, here's the actual word count example which should compile:

https://github.com/NICTA/scoobi/tree/master/examples/wordCount

On Mon, Jul 15, 2013 at 4:48 PM, Dino Fancellu notifications@github.com wrote:

persist(counts.toTextFile(args(1))

needs an extra ) at the end

Once that is done I find another compile error.

type mismatch; [error] found : (Int, Int) => Int [error] required: com.nicta.scoobi.core.Reduction[?] [error] .combine((a: Int, b: Int) => a + b) [error] ^

At the end it points to WordCount in the examples, which is also a 404 error.

Its not a good start for a "quick start"

— Reply to this email directly or view it on GitHub.

fancellu commented 11 years ago

Thanks, with the import Reduction._ it works fine.

Dino.

On 16 July 2013 00:59, Eric Springer notifications@github.com wrote:

The API change btw is: .combine((a: Int, b: Int) => a + b)

is now

.combine(Sum.int)

On Mon, Jul 15, 2013 at 4:57 PM, Eric Springer ericwspringer@gmail.com wrote:

There were some API changes which broke compile (so it'll need to be updated), but in the mean time before it and the link is fixed, here's the actual word count example which should compile:

https://github.com/NICTA/scoobi/tree/master/examples/wordCount

On Mon, Jul 15, 2013 at 4:48 PM, Dino Fancellu notifications@github.com

wrote:

persist(counts.toTextFile(args(1))

needs an extra ) at the end

Once that is done I find another compile error.

type mismatch; [error] found : (Int, Int) => Int [error] required: com.nicta.scoobi.core.Reduction[?] [error] .combine((a: Int, b: Int) => a + b) [error] ^

At the end it points to WordCount in the examples, which is also a 404 error.

Its not a good start for a "quick start"

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHubhttps://github.com/NICTA/scoobi/issues/284#issuecomment-21012682 .

etorreborre commented 11 years ago

Sorry I'm breaking it all the time. This time I broke some links because I actually simplified the build with no need to distinguish between cdh4 and cdh3 artifacts. I'll fix that later on today.

etorreborre commented 11 years ago

I'll actually fix that in the next 0.7.2 version which will contain 2 important fixes (#282 and #283).

etorreborre commented 11 years ago

fixed in 0.7.2