Dandandan commented on code in PR #2885:
URL: https://github.com/apache/arrow-datafusion/pull/2885#discussion_r919263679


##########
datafusion/optimizer/src/optimizer.rs:
##########
@@ -44,15 +44,23 @@ pub struct OptimizerConfig {
     /// Query execution start time that can be used to rewrite expressions 
such as `now()`
     /// to use a literal value instead
     pub query_execution_start_time: DateTime<Utc>,
+    next_id: usize,
 }
 
 impl OptimizerConfig {
     /// Create optimizer config
     pub fn new() -> Self {
         Self {
             query_execution_start_time: chrono::Utc::now(),
+            next_id: 1,

Review Comment:
   This is used for creating a unique alias nr. right? Maybe we can add a 
comment that it serves this purpose.



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to