mirror of
https://github.com/apache/superset.git
synced 2026-05-09 18:05:52 +00:00
6f69212f28
* feat(api): annotations and layers CRUD REST API * lint * annotations API and tests * fix openapi spec * fix openapi spec * fix openapi spec * annotations bulk delete * fix openapi spec * fix older tests * fix older tests * small lint fixes * layer_id to pk to keep broad coherence on openapi spec * fix openapi spec * one more test and validation * fix test name * fix test * fix bulk delete * add name validation * annotation uniqueness validation * lint * add sorting reqs and tests * add statsd metrics * Update superset/annotation_layers/annotations/dao.py Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com> * Update tests/core_tests.py Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com> * address comments * address my comment :) * fix, address comments * lint Co-authored-by: riahk <moar.riah@gmail.com> Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com>
71 lines
2.6 KiB
Python
71 lines
2.6 KiB
Python
# Licensed to the Apache Software Foundation (ASF) under one
|
|
# or more contributor license agreements. See the NOTICE file
|
|
# distributed with this work for additional information
|
|
# regarding copyright ownership. The ASF licenses this file
|
|
# to you under the Apache License, Version 2.0 (the
|
|
# "License"); you may not use this file except in compliance
|
|
# with the License. You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing,
|
|
# software distributed under the License is distributed on an
|
|
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
|
# KIND, either express or implied. See the License for the
|
|
# specific language governing permissions and limitations
|
|
# under the License.
|
|
import logging
|
|
from typing import Any, Dict, List, Optional
|
|
|
|
from flask_appbuilder.models.sqla import Model
|
|
from flask_appbuilder.security.sqla.models import User
|
|
from marshmallow import ValidationError
|
|
|
|
from superset.annotation_layers.commands.exceptions import (
|
|
AnnotationLayerInvalidError,
|
|
AnnotationLayerNameUniquenessValidationError,
|
|
AnnotationLayerNotFoundError,
|
|
AnnotationLayerUpdateFailedError,
|
|
)
|
|
from superset.annotation_layers.dao import AnnotationLayerDAO
|
|
from superset.commands.base import BaseCommand
|
|
from superset.dao.exceptions import DAOUpdateFailedError
|
|
from superset.models.annotations import AnnotationLayer
|
|
|
|
logger = logging.getLogger(__name__)
|
|
|
|
|
|
class UpdateAnnotationLayerCommand(BaseCommand):
|
|
def __init__(self, user: User, model_id: int, data: Dict[str, Any]):
|
|
self._actor = user
|
|
self._model_id = model_id
|
|
self._properties = data.copy()
|
|
self._model: Optional[AnnotationLayer] = None
|
|
|
|
def run(self) -> Model:
|
|
self.validate()
|
|
try:
|
|
annotation_layer = AnnotationLayerDAO.update(self._model, self._properties)
|
|
except DAOUpdateFailedError as ex:
|
|
logger.exception(ex.exception)
|
|
raise AnnotationLayerUpdateFailedError()
|
|
return annotation_layer
|
|
|
|
def validate(self) -> None:
|
|
exceptions: List[ValidationError] = list()
|
|
name = self._properties.get("name", "")
|
|
self._model = AnnotationLayerDAO.find_by_id(self._model_id)
|
|
|
|
if not self._model:
|
|
raise AnnotationLayerNotFoundError()
|
|
|
|
if not AnnotationLayerDAO.validate_update_uniqueness(
|
|
name, layer_id=self._model_id
|
|
):
|
|
exceptions.append(AnnotationLayerNameUniquenessValidationError())
|
|
|
|
if exceptions:
|
|
exception = AnnotationLayerInvalidError()
|
|
exception.add_list(exceptions)
|
|
raise exception
|