Add more sanity check to trace_length writing

Reviewed By: alambert

Differential Revision: D63497450

fbshipit-source-id: 97ffa7de47a9056d58f085de08c16cb9487eb5ae

sapp/pipeline/model_generator.py

@@ -448,8 +448,14 @@ def _generate_raw_trace_frame(
         leaf_records = []
         leaf_mapping_ids: Set[LeafMapping] = set()
         for leaf, depth in leaves:
-            if not isinstance(depth, int) or depth < 0:
-                depth = None  # avoid writing bad stuff to DB
+            # avoid writing bad trace_length to DB
+            if isinstance(depth, int):
+                if depth < 0:
+                    depth = 0
+                elif depth > 1000:
+                    depth = 1000
+            else:
+                depth = None
             leaf_record = self._get_shared_text(leaf_kind, leaf)
             caller_leaf_id = self.graph.get_transform_normalized_caller_kind_id(
                 leaf_record