Fix coding errors (#149)
parent
bb49172d92
commit
fb6624215a
|
@ -35,7 +35,7 @@ class SpendingByCategory(MRJob):
|
||||||
if period == self.current_year_month():
|
if period == self.current_year_month():
|
||||||
yield (period, category), amount
|
yield (period, category), amount
|
||||||
|
|
||||||
def reducer(self, key, value):
|
def reducer(self, key, values):
|
||||||
"""Sum values for each key.
|
"""Sum values for each key.
|
||||||
|
|
||||||
(2016-01, shopping), 125
|
(2016-01, shopping), 125
|
||||||
|
|
|
@ -26,7 +26,7 @@ class HitCounts(MRJob):
|
||||||
period = self.extract_year_month(line)
|
period = self.extract_year_month(line)
|
||||||
yield (period, url), 1
|
yield (period, url), 1
|
||||||
|
|
||||||
def reducer(self, key, value):
|
def reducer(self, key, values):
|
||||||
"""Sum values for each key.
|
"""Sum values for each key.
|
||||||
|
|
||||||
(2016-01, url0), 2
|
(2016-01, url0), 2
|
||||||
|
|
|
@ -9,7 +9,7 @@ class SalesRanker(MRJob):
|
||||||
"""Return True if timestamp is within past week, False otherwise."""
|
"""Return True if timestamp is within past week, False otherwise."""
|
||||||
...
|
...
|
||||||
|
|
||||||
def mapper(self, _ line):
|
def mapper(self, _, line):
|
||||||
"""Parse each log line, extract and transform relevant lines.
|
"""Parse each log line, extract and transform relevant lines.
|
||||||
|
|
||||||
Emit key value pairs of the form:
|
Emit key value pairs of the form:
|
||||||
|
@ -25,7 +25,7 @@ class SalesRanker(MRJob):
|
||||||
if self.within_past_week(timestamp):
|
if self.within_past_week(timestamp):
|
||||||
yield (category, product_id), quantity
|
yield (category, product_id), quantity
|
||||||
|
|
||||||
def reducer(self, key, value):
|
def reducer(self, key, values):
|
||||||
"""Sum values for each key.
|
"""Sum values for each key.
|
||||||
|
|
||||||
(foo, p1), 2
|
(foo, p1), 2
|
||||||
|
@ -74,4 +74,4 @@ class SalesRanker(MRJob):
|
||||||
|
|
||||||
|
|
||||||
if __name__ == '__main__':
|
if __name__ == '__main__':
|
||||||
HitCounts.run()
|
SalesRanker.run()
|
||||||
|
|
Loading…
Reference in New Issue