[
https://issues.apache.org/jira/browse/BEAM-9561?focusedWorklogId=483209&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-483209
]
ASF GitHub Bot logged work on BEAM-9561:
----------------------------------------
Author: ASF GitHub Bot
Created on: 12/Sep/20 21:32
Start Date: 12/Sep/20 21:32
Worklog Time Spent: 10m
Work Description: y1chi commented on a change in pull request #12815:
URL: https://github.com/apache/beam/pull/12815#discussion_r486672602
##########
File path: sdks/python/apache_beam/dataframe/doctests.py
##########
@@ -319,25 +319,24 @@ def run(self, test, **kwargs):
# Don't fail doctests that raise this error.
example.exc_msg = (
'apache_beam.dataframe.frame_base.WontImplementError: ...')
- self.wont_implement += 1
with self._test_env.context():
result = super(BeamDataframeDoctestRunner, self).run(test, **kwargs)
return result
def report_success(self, out, test, example, got):
def extract_concise_reason(got):
- m = re.search(r"(WontImplementError:.*)\n$", got)
+ m = re.search(r"WontImplementError:\s+(.*)\n$", got)
if m:
return m.group(1)
elif "NameError" in got:
- return "NameError"
+ return "NameError following WontImplementError"
Review comment:
I assume all the errors here are under the precondition that
WontImplementError is seen thus even the regex is not matched we know it is
following WontImplementError?
##########
File path: sdks/python/apache_beam/dataframe/doctests.py
##########
@@ -319,25 +319,24 @@ def run(self, test, **kwargs):
# Don't fail doctests that raise this error.
example.exc_msg = (
'apache_beam.dataframe.frame_base.WontImplementError: ...')
- self.wont_implement += 1
with self._test_env.context():
result = super(BeamDataframeDoctestRunner, self).run(test, **kwargs)
return result
def report_success(self, out, test, example, got):
def extract_concise_reason(got):
- m = re.search(r"(WontImplementError:.*)\n$", got)
+ m = re.search(r"WontImplementError:\s+(.*)\n$", got)
if m:
return m.group(1)
elif "NameError" in got:
- return "NameError"
+ return "NameError following WontImplementError"
Review comment:
I assume all the errors here are under the precondition that
WontImplementError is seen thus even the regex is not matched we know it is
following WontImplementError?
##########
File path: sdks/python/apache_beam/dataframe/doctests.py
##########
@@ -319,25 +319,24 @@ def run(self, test, **kwargs):
# Don't fail doctests that raise this error.
example.exc_msg = (
'apache_beam.dataframe.frame_base.WontImplementError: ...')
- self.wont_implement += 1
with self._test_env.context():
result = super(BeamDataframeDoctestRunner, self).run(test, **kwargs)
return result
def report_success(self, out, test, example, got):
def extract_concise_reason(got):
- m = re.search(r"(WontImplementError:.*)\n$", got)
+ m = re.search(r"WontImplementError:\s+(.*)\n$", got)
if m:
return m.group(1)
elif "NameError" in got:
- return "NameError"
+ return "NameError following WontImplementError"
Review comment:
I assume all the errors here are under the precondition that
WontImplementError is seen thus even the regex is not matched we know it is
following WontImplementError?
##########
File path: sdks/python/apache_beam/dataframe/doctests.py
##########
@@ -319,25 +319,24 @@ def run(self, test, **kwargs):
# Don't fail doctests that raise this error.
example.exc_msg = (
'apache_beam.dataframe.frame_base.WontImplementError: ...')
- self.wont_implement += 1
with self._test_env.context():
result = super(BeamDataframeDoctestRunner, self).run(test, **kwargs)
return result
def report_success(self, out, test, example, got):
def extract_concise_reason(got):
- m = re.search(r"(WontImplementError:.*)\n$", got)
+ m = re.search(r"WontImplementError:\s+(.*)\n$", got)
if m:
return m.group(1)
elif "NameError" in got:
- return "NameError"
+ return "NameError following WontImplementError"
Review comment:
I assume all the errors here are under the precondition that
WontImplementError is seen thus even the regex is not matched we know it is
following WontImplementError?
##########
File path: sdks/python/apache_beam/dataframe/doctests.py
##########
@@ -319,25 +319,24 @@ def run(self, test, **kwargs):
# Don't fail doctests that raise this error.
example.exc_msg = (
'apache_beam.dataframe.frame_base.WontImplementError: ...')
- self.wont_implement += 1
with self._test_env.context():
result = super(BeamDataframeDoctestRunner, self).run(test, **kwargs)
return result
def report_success(self, out, test, example, got):
def extract_concise_reason(got):
- m = re.search(r"(WontImplementError:.*)\n$", got)
+ m = re.search(r"WontImplementError:\s+(.*)\n$", got)
if m:
return m.group(1)
elif "NameError" in got:
- return "NameError"
+ return "NameError following WontImplementError"
Review comment:
I assume all the errors here are under the precondition that
WontImplementError is seen thus even the regex is not matched we know it is
following WontImplementError?
##########
File path: sdks/python/apache_beam/dataframe/doctests.py
##########
@@ -319,25 +319,24 @@ def run(self, test, **kwargs):
# Don't fail doctests that raise this error.
example.exc_msg = (
'apache_beam.dataframe.frame_base.WontImplementError: ...')
- self.wont_implement += 1
with self._test_env.context():
result = super(BeamDataframeDoctestRunner, self).run(test, **kwargs)
return result
def report_success(self, out, test, example, got):
def extract_concise_reason(got):
- m = re.search(r"(WontImplementError:.*)\n$", got)
+ m = re.search(r"WontImplementError:\s+(.*)\n$", got)
if m:
return m.group(1)
elif "NameError" in got:
- return "NameError"
+ return "NameError following WontImplementError"
Review comment:
I assume all the errors here are under the precondition that
WontImplementError is seen thus even the regex is not matched we know it is
following WontImplementError?
##########
File path: sdks/python/apache_beam/dataframe/doctests.py
##########
@@ -319,25 +319,24 @@ def run(self, test, **kwargs):
# Don't fail doctests that raise this error.
example.exc_msg = (
'apache_beam.dataframe.frame_base.WontImplementError: ...')
- self.wont_implement += 1
with self._test_env.context():
result = super(BeamDataframeDoctestRunner, self).run(test, **kwargs)
return result
def report_success(self, out, test, example, got):
def extract_concise_reason(got):
- m = re.search(r"(WontImplementError:.*)\n$", got)
+ m = re.search(r"WontImplementError:\s+(.*)\n$", got)
if m:
return m.group(1)
elif "NameError" in got:
- return "NameError"
+ return "NameError following WontImplementError"
Review comment:
I assume all the errors here are under the precondition that
WontImplementError is seen thus even the regex is not matched we know it is
following WontImplementError?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 483209)
Time Spent: 8h 40m (was: 8.5h)
> Run pandas tests with Beam Dataframe API
> ----------------------------------------
>
> Key: BEAM-9561
> URL: https://issues.apache.org/jira/browse/BEAM-9561
> Project: Beam
> Issue Type: Improvement
> Components: sdk-py-core
> Reporter: Brian Hulette
> Assignee: Robert Bradshaw
> Priority: P2
> Fix For: Not applicable
>
> Time Spent: 8h 40m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
