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

Posted to dev@hop.apache.org by GitBox <gi...@apache.org> on 2020/12/30 19:50:36 UTC

[GitHub] [incubator-hop] hansva edited a comment on pull request #427: Hop 2217

hansva edited a comment on pull request #427:
URL: https://github.com/apache/incubator-hop/pull/427#issuecomment-752737860


   I stopped halfway this needs quite a lot of changes before we can consider merging:
   2 more points that worry me:
   
   - are all new included libraries a compatible license?
   - we should either create a database type for cassandra to store the metadata, or create a new metadata type to store the cassandra info, I do not like that all the fields are in the transform every time (prefered would be the second as regular database transforms can not use this meta)


----------------------------------------------------------------
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:
users@infra.apache.org