You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@gobblin.apache.org by GitBox <gi...@apache.org> on 2022/03/26 00:53:59 UTC

[GitHub] [gobblin] arjun4084346 commented on pull request #3484: [GOBBLIN-1627] provide option to convert datanodes names

arjun4084346 commented on pull request #3484:
URL: https://github.com/apache/gobblin/pull/3484#issuecomment-1079539061


   Ah actually missed that comment. 
   Sometimes the flowgraph has old names of data node. e.g. LinkedIn's internal flowgraph has name griduse, but the correct name used by Azkaban was gridmem1e. Both actually mean the same node. But if we change the node, it will break old customer's code, if we add a duplicate node, it will be a tech debt to slowly deprecate the old name. I made a client side change, where it would change the name, but @Will-Lo  suggested that it would be better if we make changes on server side, because change here will be applicable to all kinds of clients and deployment will be a less step process.


-- 
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: dev-unsubscribe@gobblin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org