alexkruc commented on code in PR #25280:
URL: https://github.com/apache/airflow/pull/25280#discussion_r938684056
##########
airflow/models/dag.py:
##########
@@ -2797,6 +2831,29 @@ def __repr__(self):
return self.name
+class DagOwnerAttributes(Base):
+ """
+ Table defining different owner attributes. For example, a link for an
owner that will be passed as
+ a hyperlink to the DAGs view
+ """
+
+ __tablename__ = "dag_owner_attributes"
+ dag_id = Column(
+ StringID(),
+ ForeignKey('dag.dag_id', name='dag.dag_id', ondelete='CASCADE'),
+ nullable=False,
+ primary_key=True,
+ )
+ owner = Column(String(100), primary_key=True, nullable=False)
Review Comment:
I changed it to 500 characters, just as the current Log table defines the
owner :) Do you think we still need to add documentation and validation? And if
so, does it need to be in this PR, or is it a separate enhancement?
https://github.com/apache/airflow/blob/main/airflow/models/log.py#L38
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
