villebro commented on a change in pull request #16991:
URL: https://github.com/apache/superset/pull/16991#discussion_r759106982



##########
File path: superset/charts/data/commands/get_data_command.py
##########
@@ -14,6 +14,8 @@
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
 # under the License.
+from __future__ import annotations
+

Review comment:
       @ofekisr My point for introducing the FF is to gradually introduce the 
new stricter security check in a way which makes it possible to catch potential 
regressions that weren't caught during initial review. By having an 
introductory period during which the stricter checks are behind a FF, orgs that 
require the new stricter checking can help harden the feature by pushing fixes 
to master before the new checks are made generally available.




-- 
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]



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

Reply via email to