Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Data Factory]Add support for data flow #7765

Merged
merged 2 commits into from
Sep 26, 2019

Conversation

jackmagic313
Copy link
Contributor

@jackmagic313 jackmagic313 commented Sep 25, 2019

  • Added support for Data Flow:
    * Added data flow object model and CRUD APIs
    * Added data flow debug session APIs
    * Added support for execute data flow activity type

Swagger PRs:
Azure/azure-rest-api-specs#7069
Azure/azure-rest-api-specs#7088
Azure/azure-rest-api-specs#7242
Azure/azure-rest-api-specs#7307

@@ -2,6 +2,10 @@

## Current version
### Feature Additions
- Added support for Data Flow:
Copy link
Contributor

Choose a reason for hiding this comment

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

You can update the version here now - add "## Version 4.2.0"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -2,6 +2,10 @@

## Current version
### Feature Additions
- Added support for Data Flow:
* Added data flow object model and CRUD APIs
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Keep whitespaces the same, copy paste from other lines

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I need the intend in order to add these as sub list

@isra-fel isra-fel added Mgmt This issue is related to a management-plane library. needs-review labels Sep 26, 2019
@isra-fel isra-fel self-assigned this Sep 26, 2019
Copy link
Member

@isra-fel isra-fel left a comment

Choose a reason for hiding this comment

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

@jackmagic313 Please add link to swagger review PR(s) in this PR's description, and see my inline comments. Thanks!

@jackmagic313
Copy link
Contributor Author

@jackmagic313 Please add link to swagger review PR(s) in this PR's description, and see my inline comments. Thanks!

Replied, please help to take a look again, thx!

@isra-fel isra-fel merged commit 7c5ce9b into Azure:master Sep 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mgmt This issue is related to a management-plane library.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants