You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by "paul-rogers (via GitHub)" <gi...@apache.org> on 2023/04/04 00:45:11 UTC

[GitHub] [druid] paul-rogers opened a new pull request, #14019: Temporarily revert the catalog for Druid 26

paul-rogers opened a new pull request, #14019:
URL: https://github.com/apache/druid/pull/14019

   As we come up on the Druid 26 release, it became clear that there are too many open issues in the catalog project to include the catalog in the release. Some issues are minor (the form of the enhanced table functions), some are major (what to do with aggregations, and whether we need to store table schema as we move to a schemaless world.)
   
   The safest bet is to temporarily remove the catalog code until we resolve the open issues. Then, we can restore the code from this PR, and go in the direction we ultimately decide on.
   
   This PR leaves some of the "new" plumbing for use by the `extern` function. The required classes were moved out of the now-reverted `catalog` package and into a `sql.calcite` package. This way, the code is consistent and clean for this release.
   
   Catalog-related tests and ITs were also removed, as were documentation references to the enhanced table functions.
   
   
   #### Release note
   
   No release notes needed because the catalog was not part of a prior release.
   
   <hr>
   
   This PR has:
   
   - [X] been self-reviewed.
   - [X] a release note entry in the PR description.
   - [ ] been tested in a test Druid cluster.
   


-- 
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: commits-unsubscribe@druid.apache.org

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


[GitHub] [druid] paul-rogers merged pull request #14019: Temporarily revert extended table functions for Druid 26

Posted by "paul-rogers (via GitHub)" <gi...@apache.org>.
paul-rogers merged PR #14019:
URL: https://github.com/apache/druid/pull/14019


-- 
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: commits-unsubscribe@druid.apache.org

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


[GitHub] [druid] paul-rogers commented on pull request #14019: Temporarily revert extended table functions for Druid 26

Posted by "paul-rogers (via GitHub)" <gi...@apache.org>.
paul-rogers commented on PR #14019:
URL: https://github.com/apache/druid/pull/14019#issuecomment-1498330947

   This PR _seems_ ready. There is a build failure, but in the always-flaky Kafka IT.


-- 
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: commits-unsubscribe@druid.apache.org

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