[
https://issues.apache.org/jira/browse/BEAM-13966?focusedWorklogId=750420&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-750420
]
ASF GitHub Bot logged work on BEAM-13966:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 30/Mar/22 20:13
Start Date: 30/Mar/22 20:13
Worklog Time Spent: 10m
Work Description: TheNeuralBit commented on a change in pull request
#17043:
URL: https://github.com/apache/beam/pull/17043#discussion_r838926287
##########
File path: sdks/python/apache_beam/dataframe/pandas_doctests_test.py
##########
@@ -876,7 +869,7 @@ def test_top_level(self):
'merge_ordered(df1, df2, fill_method="ffill", left_by="group")'
],
# Expected error.
- 'pivot': ["df.pivot(index='foo', columns='bar', values='baz')"],
+ 'pivot': ["*"],
Review comment:
I don't think we want to skip all the pivot tests, can we revert this
line and instead add `'pivot': ["*"]` to wont_implement_ok (as suggested above).
##########
File path: sdks/python/apache_beam/dataframe/pandas_doctests_test.py
##########
@@ -366,9 +362,7 @@ def test_dataframe_tests(self):
# actually raise NotImplementedError
'pandas.core.frame.DataFrame.pivot_table': ['*'],
# Expected to raise a ValueError, but we raise NotImplementedError
- 'pandas.core.frame.DataFrame.pivot': [
- "df.pivot(index='foo', columns='bar', values='baz')"
- ],
Review comment:
Similarly here
##########
File path: sdks/python/apache_beam/dataframe/pandas_doctests_test.py
##########
@@ -832,7 +826,6 @@ def test_top_level(self):
'melt': ['*'],
'merge': ["df1.merge(df2, how='cross')"],
'merge_asof': ['*'],
- 'pivot': ['*'],
Review comment:
I think this should be wont_implement_ok instead, right?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 750420)
Time Spent: 4.5h (was: 4h 20m)
> Implement DataFrame.pivot() for DataFrame API
> ---------------------------------------------
>
> Key: BEAM-13966
> URL: https://issues.apache.org/jira/browse/BEAM-13966
> Project: Beam
> Issue Type: Sub-task
> Components: dsl-dataframe, sdk-py-core
> Reporter: Andy Ye
> Assignee: Andy Ye
> Priority: P3
> Labels: dataframe-api
> Time Spent: 4.5h
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
