[DISCUSSION] Should we implement access control hooks in DataFusion?
- Lenguaje dominante
- Rust
- Estrellas
- 9.3k
- Forks
- 2.4k
- Merge medio
- 3 d 11 h
- PR fusionados (30 d)
- 360
Descripción
### Is your feature request related to a problem or challenge?
@comphead brings up an excellent point here: https://github.com/apache/arrow-datafusion/pull/7441#issuecomment-1698341294
Basically many database systems allow some sort of access control such as allowing some users the ability to read data but restricting writing.
There are many different granularities of such controls (like restricted to schemas, tables, read schema vs read data, etc) that a system might want to implement.
### Describe the solution you'd like
If anyone needs this today, they can build it on top of `LogicalPlan` (by checking `LogicalPlan` contents and implementing whatever controls they want).
It might be interesting to add some sort of built in (extensible) mechanism into DataFusion to make this process easier.
### Describe alternatives you've considered
I think we should wait for someone with a real usecase / implementation on top of `LogicalPlan` that we can upstream once we work out the details rather than designing this in advance but I wanted to file the ticket to track the idea
### Additional context
_No response_
Guía de contribución
Línea de trabajo
Empieza leyendo el comentario enlazado de PR #7441 y la implementación de LogicalPlan para entender el caso de uso propuesto para el control de acceso. Darlo por terminado requeriría un alcance concreto y acordado, además de un mecanismo extensible, pero esta discusión pospone actualmente el diseño hasta que haya una implementación real o un caso de uso disponible.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- rust, sql
- Área
- databases, security
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100