You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@seatunnel.apache.org by GitBox <gi...@apache.org> on 2022/01/02 06:36:16 UTC

[GitHub] [incubator-seatunnel] leo65535 opened a new issue #924: [Bug] [config] Can not resolve variables.

leo65535 opened a new issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924


   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/incubator-seatunnel/issues?q=is%3Aissue+label%3A%22bug%22) and found no similar issues.
   
   
   ### What happened
   
   ### Conf
   ```
   filter {
   
     sql {
       table_name = "user_view"
       sql = "select '"${city2}"'"
       result_table_name = "result1"
     }
   
     sql {
       table_name = "user_view"
       sql = "select '"${dt}"'"
       result_table_name = "result2"
     }
   }
   
   ```
   
   ### org.apache.seatunnel.config.CompleteTests
   
   ```
   
   public static void main(String[] args) {
   
     System.out.println("Hello World");
     System.setProperty("dt", "20190318");
     System.setProperty("city2", "shanghai");
   
     CompleteTests completeTests = new CompleteTests();
     Config config = ConfigFactory.parseFile(completeTests.getFileFromResources("seatunnel/variables.conf"));
     config = config.resolve(ConfigResolveOptions.defaults().setAllowUnresolved(true))
       .resolveWith(ConfigFactory.systemProperties(), ConfigResolveOptions.defaults().setAllowUnresolved(true));
   
     ConfigRenderOptions renderOptions = ConfigRenderOptions.defaults().setComments(false).setOriginComments(false);
     System.out.println(config.root().render(renderOptions));
       
   }
   ```
   
   ### Result
   
   ![image](https://user-images.githubusercontent.com/95013770/147868248-0a94c3e0-c8c8-471c-9841-00e356f901e9.png)
   
   
   
   
   ### SeaTunnel Version
   
   dev
   
   ### SeaTunnel Config
   
   ```conf
   dd
   ```
   
   
   ### Running Command
   
   ```shell
   ddd
   ```
   
   
   ### Error Exception
   
   ```log
   ddd
   ```
   
   
   ### Flink or Spark Version
   
   dd
   
   ### Java or Scala Version
   
   dd
   
   ### Screenshots
   
   ddd
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] leo65535 removed a comment on issue #924: [Bug] [config] Can not resolve variables.

Posted by GitBox <gi...@apache.org>.
leo65535 removed a comment on issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924#issuecomment-1004469191


   And, I also found the commit of the problem, it is `5c44cb64`


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] leo65535 commented on issue #924: [Bug] [config] Can not resolve variables.

Posted by GitBox <gi...@apache.org>.
leo65535 commented on issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924#issuecomment-1004468644


   hi @garyelephant, I use git history for `CompleteTests` file.
   
   ![image](https://user-images.githubusercontent.com/95013770/147998599-8bdfaa23-f27d-4136-a144-384acbf5adc5.png)
   


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] leo65535 commented on issue #924: [Bug] [config] Can not resolve variables.

Posted by GitBox <gi...@apache.org>.
leo65535 commented on issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924#issuecomment-1004469191


   And, I also found the commit of the problem, it is `5c44cb64`


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] leo65535 edited a comment on issue #924: [Bug] [config] Can not resolve variables.

Posted by GitBox <gi...@apache.org>.
leo65535 edited a comment on issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924#issuecomment-1004491395


   will fixed at https://github.com/apache/incubator-seatunnel/pull/906


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] garyelephant commented on issue #924: [Bug] [config] Can not resolve variables.

Posted by GitBox <gi...@apache.org>.
garyelephant commented on issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924#issuecomment-1003803757


   @leo65535 did you mean that codes in dev branch have a bug of failing to parse variable substitution? why you compare dev branch with commit `1faf2e` ?


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] leo65535 closed issue #924: [Bug] [config] Can not resolve variables.

Posted by GitBox <gi...@apache.org>.
leo65535 closed issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924


   


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] leo65535 edited a comment on issue #924: [Bug] [config] Can not resolve variables.

Posted by GitBox <gi...@apache.org>.
leo65535 edited a comment on issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924#issuecomment-1004468644


   hi @garyelephant, I use git history for `CompleteTests` file.
   
   And, I also found the commit of the problem, it is `5c44cb64`
   
   ![image](https://user-images.githubusercontent.com/95013770/147998599-8bdfaa23-f27d-4136-a144-384acbf5adc5.png)
   


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] leo65535 commented on issue #924: [Bug] [config] Can not resolve variables.

Posted by GitBox <gi...@apache.org>.
leo65535 commented on issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924#issuecomment-1004491395


   fixed at https://github.com/apache/incubator-seatunnel/pull/906


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] leo65535 edited a comment on issue #924: [Bug] [config] Can not resolve variables.

Posted by GitBox <gi...@apache.org>.
leo65535 edited a comment on issue #924:
URL: https://github.com/apache/incubator-seatunnel/issues/924#issuecomment-1004468644


   hi @garyelephant, I use git history for `CompleteTests` file.
   
   And, I also found the commit of the problem, it is `5c44cb64`. After `5c44cb64`, config can not resolve variables.
   
   
   
   <br>
   
   ![image](https://user-images.githubusercontent.com/95013770/147998599-8bdfaa23-f27d-4136-a144-384acbf5adc5.png)
   


-- 
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: commits-unsubscribe@seatunnel.apache.org

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