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/09/10 22:01:26 UTC

[GitHub] [druid] clintropolis commented on pull request #10370: add computed Expr output types

clintropolis commented on pull request #10370:
URL: https://github.com/apache/druid/pull/10370#issuecomment-690755046


   This PR will cause soft conflicts (which will probably not show up as merge conflicts) with any open PRs that are adding any new `Expr`, `ExprMacro`, `Function`, `ApplyFunction`, since they will need to include a `getOutputType` implementation after this PR is merged.
   
   at minimum these open PRs would be affected:
   #10084
   #10230
   #10350
   
   and likewise if any of these go in I will need to fixup this branch before it can be merged.


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



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