search

citmp2015 / flink-visual-programming

2 stars 9 forks source link

JSON parser #92

Closed fb55 closed 8 years ago

fb55 commented 8 years ago

I figure I should open a PR for this :)

This should be enough to complete the word count example. The current structure for components isn’t easy to map to the JSON format, we should talk about it on Wednesday.

Fabsi110 commented 8 years ago

Laut Jenkins kompiliert der PageRankTest nicht... @fb55 kannst du da nochmal kurz nachgucken?

fb55 commented 8 years ago

Der PR war eigentlich für modeltmp, der Fehler kam aus der Branch. Für den Parser relevant sind nur die letzten drei Commits.

Fabsi110 commented 8 years ago

Aha, kannst du den PR dann nochmal neu stellen mit nur den letzten 3 Commits? Und mir fällt gerade auf, dass du nicht in dev, sondern in den mastern mergen willst. Die PR's werden aber nur in den dev gemergt, nicht in den master...

fb55 commented 8 years ago

Wie gesagt, sollte eigentlich in modeltmp gemerged werden, weil's auch auf der Branch basiert. Versuche gerade einen neuen PR zu erstellen (hat geklappt: #95).