Skip to content

Conversation

@Nuttymoon
Copy link
Contributor

Which issue(s) this PR fixes:

Fixes #277

Additional comments:

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004 and that you give permission to TOSIT,
you have to acknowledge this by using the following check-box.

@gboutry
Copy link
Contributor

gboutry commented Aug 30, 2022

@Nuttymoon
Copy link
Contributor Author

There are still references to Oozie in https://github.com/TOSIT-IO/tdp-collection/blob/277-remove-oozie/ARCHITECTURE.md

Yes, I know. As this is an example I am not sure that it is armful. If we want to remove it we should regenerate the graph. What tool was used?

@gboutry
Copy link
Contributor

gboutry commented Aug 30, 2022

There are still references to Oozie in https://github.com/TOSIT-IO/tdp-collection/blob/277-remove-oozie/ARCHITECTURE.md

Yes, I know. As this is an example I am not sure that it is armful. If we want to remove it we should regenerate the graph. What tool was used?

I have no idea, I'm not even sure a tool was used

Copy link
Contributor

@gboutry gboutry left a comment

Choose a reason for hiding this comment

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

@Nuttymoon but anyway, this document is outdated, especially this part of the document. Therefore, we can solve this in another issue.

@Nuttymoon Nuttymoon merged commit 5878507 into master Aug 30, 2022
@Nuttymoon Nuttymoon deleted the 277-remove-oozie branch August 30, 2022 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove Oozie.

4 participants