Skip to content
This repository has been archived by the owner on Feb 8, 2019. It is now read-only.

[GEARPUMP-192] refactor example sources task to use DataSourceAPI #78

Closed
wants to merge 8 commits into from

Conversation

Roshanson
Copy link
Contributor

@Roshanson Roshanson commented Aug 26, 2016

[GEARPUMP-192] Refactor example sources task to use DataSourceAPI.

@manuzhang
Copy link
Contributor

please modify the commit comment to the format "[GEARPUMP-XXX] xyz"

@manuzhang
Copy link
Contributor

also, please fix the UT failure

@Roshanson Roshanson changed the title refactor example sources task to use DataSourceAPI [GEARPUMP-192] refactor example sources task to use DataSourceAPI Aug 29, 2016
override def onStart(startTime: Instant): Unit = {
self ! Watermark(Instant.now)
}
class Split() extends DataSource {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I prefer class Split extends DataSource, no braces.

@codecov-io
Copy link

codecov-io commented Sep 2, 2016

Current coverage is 70.08% (diff: 100%)

Merging #78 into master will increase coverage by 1.48%

@@             master        #78   diff @@
==========================================
  Files           177        177          
  Lines          5927       5927          
  Methods        5604       5606     +2   
  Messages          0          0          
  Branches        323        321     -2   
==========================================
+ Hits           4066       4154    +88   
+ Misses         1861       1773    -88   
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 529799c...f4c6737

@manuzhang
Copy link
Contributor

+1

@asfgit asfgit closed this in a743a9c Sep 2, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants