jihoonson commented on issue #9421: Unknown complex types break Druid SQL
URL: https://github.com/apache/druid/issues/9421#issuecomment-610548834
 
 
   AFAIK, we have different compatibility policies in different areas. There 
are two areas I'm aware of.
   
   - For APIs, we have `PublicApi` and `ExtensionPoint` annotations. For those 
APIs annotated with one of them, there could be incompatible changes in those 
APIs between major releases. For non-annotated APIs, they can change in 
breaking ways in any releases. 
   - For segment read, we should be able to read any format of segments no 
matter how old Druid created them (I didn't know this policy when I raised 
https://github.com/apache/druid/pull/9246). This applies to segment format, all 
shardSpecs, and all loadSpecs. 
   
   These policies apply to across all Druid modules including contrib 
extensions. 
   
   Based on these policies, if the aggregator type has changed inadvertently, 
that's a compatibility bug and should be fixed by supporting both old and new 
types.
   
   However, I think your PR might be still useful since people sometimes might 
want to run Druid without some extensions (for example, they just want to stop 
using those extensions). Even in that case, as you mentioned, it would be nice 
that Druid could be still functioning. Your PR could address this use case. 
I'll review it soon.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to