You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/01/22 23:45:22 UTC

[GitHub] [druid] zachjsh opened a new issue #9240: Bring Azure functionality to core functionality

zachjsh opened a new issue #9240: Bring Azure functionality to core functionality
URL: https://github.com/apache/druid/issues/9240
 
 
   ### Motivation
   
   There is a significant potential Azure user base interested in using or already using Druid. Bringing Azure functionality to core will make druid easier to use, and will help ensure the quality of this functionality.
   
   ### Proposed changes
   
   1. Move the azure-extension into extensions-core, we are willing to put effort into maintaining this extension as time goes on.
   2. Offically deprecate firehose functionality for azure, and implement a new input source mechanism. firehose implementation will not be maintained nor tested.
   

----------------------------------------------------------------
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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] jihoonson commented on issue #9240: Bring Azure functionality to core functionality

Posted by GitBox <gi...@apache.org>.
jihoonson commented on issue #9240: Bring Azure functionality to core functionality
URL: https://github.com/apache/druid/issues/9240#issuecomment-577440452
 
 
   @zachjsh thanks for bringing this up! I like the idea. Would you please fill out the missing sections in the proposal? Rationale and Operational impact sections are mandatory and I think this proposal should include the Test plan section as well. You can leave some sections empty if you don't think anything is required, but I believe you have somethings to write for each section.

----------------------------------------------------------------
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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] drcrallen commented on issue #9240: Bring Azure functionality to core functionality

Posted by GitBox <gi...@apache.org>.
drcrallen commented on issue #9240: Bring Azure functionality to core functionality
URL: https://github.com/apache/druid/issues/9240#issuecomment-577785675
 
 
   Is there anything that needs to go under https://github.com/apache/druid/tree/master/cloud to support Azure?

----------------------------------------------------------------
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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] jihoonson commented on issue #9240: Bring Azure functionality to core functionality

Posted by GitBox <gi...@apache.org>.
jihoonson commented on issue #9240: Bring Azure functionality to core functionality
URL: https://github.com/apache/druid/issues/9240#issuecomment-577458252
 
 
   @zachjsh thanks for the update. :slightly_smiling_face:

----------------------------------------------------------------
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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] zachjsh commented on issue #9240: Bring Azure functionality to core functionality

Posted by GitBox <gi...@apache.org>.
zachjsh commented on issue #9240: Bring Azure functionality to core functionality
URL: https://github.com/apache/druid/issues/9240#issuecomment-578255657
 
 
   @drcrallen , i dont think that anything is needed in https://github.com/apache/druid/tree/master/cloud. This directory confuses me, it seems to be for common code, but there doesnt seem to be any code shared here. There are just specific implementations of cloud provider configs and modules. I think that should live in the extension rather than here.

----------------------------------------------------------------
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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] zachjsh edited a comment on issue #9240: Bring Azure functionality to core functionality

Posted by GitBox <gi...@apache.org>.
zachjsh edited a comment on issue #9240: Bring Azure functionality to core functionality
URL: https://github.com/apache/druid/issues/9240#issuecomment-578255657
 
 
   @drcrallen , Correct we will need to add common classes like credentials and other configuration that may be shared across multiple Azure services in that directory. 

----------------------------------------------------------------
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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org