You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@linkis.apache.org by GitBox <gi...@apache.org> on 2021/12/22 06:56:28 UTC
[GitHub] [incubator-linkis] saLeox opened a new pull request #1232: Use current user to read result file, and abstract the file permission in filesystem
saLeox opened a new pull request #1232:
URL: https://github.com/apache/incubator-linkis/pull/1232
### What is the purpose of the change
For now, the file permission of the generated result set in HDFS or local file system is not specified in one place globally, and followed the default value (744), and JVM user is used to read those result file instead of current login user.
It causes two issues:
1) The result set is transparent to all hdfs users;
2) JVM user cannot read current login user's result set when the permission of file is changed to more strict value(like 740).
### Brief change log
- Specify the permission value in FileSystem (740 for file and 750 for folder) , and apply it in in local file system and hdfs;
- Use current login user to read result file instead of JVM user.
### Verifying this change
(Please pick either of the following options)
This change added tests and can be verified as follows:
- Submit any type of task normally, and check whether of permission value of results set or log in the hdfs is 740, and it's expected to be available to current login user.
### Does this pull request potentially affect one of the following parts:
- Dependencies (does it add or upgrade a dependency): (no)
- Anything that affects deployment: (no)
- The MGS(Microservice Governance Services), i.e., Spring Cloud Gateway, OpenFeign, Eureka.: (no)
### Documentation
- Does this pull request introduce a new feature? (no)
--
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: dev-unsubscribe@linkis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org
For additional commands, e-mail: dev-help@linkis.apache.org
[GitHub] [incubator-linkis] peacewong merged pull request #1232: Use current user to read result file, and abstract the file permission in filesystem
Posted by GitBox <gi...@apache.org>.
peacewong merged pull request #1232:
URL: https://github.com/apache/incubator-linkis/pull/1232
--
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: dev-unsubscribe@linkis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org
For additional commands, e-mail: dev-help@linkis.apache.org
[GitHub] [incubator-linkis] peacewong merged pull request #1232: Use current user to read result file, and abstract the file permission in filesystem
Posted by GitBox <gi...@apache.org>.
peacewong merged pull request #1232:
URL: https://github.com/apache/incubator-linkis/pull/1232
--
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: dev-unsubscribe@linkis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org
For additional commands, e-mail: dev-help@linkis.apache.org