Rick Cheng

Results 40 comments of Rick Cheng
trafficstars

Hi, @EricGao888 , I'm interested in this issue. If help is needed, I am willing to participate in it.

Hi, @caishunfeng , I've moved `SensitiveDataConverter` to `dolphinscheduler-common` module. cc @zhongjiajie

> I have tested this PR in my dev cluster, it works well and the issue has been fixed. Thanks @cqutwangyu . > > What should be the next step...

> attach test pics: ![image](https://private-user-images.githubusercontent.com/13224827/309485479-ae1e3474-bb2a-45b7-86b6-6963e3cfc9ab.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MDk1MjU1MzAsIm5iZiI6MTcwOTUyNTIzMCwicGF0aCI6Ii8xMzIyNDgyNy8zMDk0ODU0NzktYWUxZTM0NzQtYmIyYS00NWI3LTg2YjYtNjk2M2UzY2ZjOWFiLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzA0VDA0MDcxMFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQ4MDFjZTI3OWVhMDZjOTIzNWM4M2U5NTBhOTlmMWI0YzFhMGMwMWE5M2ZiZDE3ZWQ1NTA2ZWNiM2RkNTZkZTMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.icqtVm_lInEN0HNPDPoGnsx2Al2luhB2vT1XWkIu34Y) ![image](https://private-user-images.githubusercontent.com/13224827/309485544-71068ac1-3dc8-4fcd-abad-e1f7e3a553a5.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MDk1MjU1MzAsIm5iZiI6MTcwOTUyNTIzMCwicGF0aCI6Ii8xMzIyNDgyNy8zMDk0ODU1NDQtNzEwNjhhYzEtM2RjOC00ZmNkLWFiYWQtZTFmN2UzYTU1M2E1LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzA0VDA0MDcxMFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTM5Y2FlN2Q5YTdiNzhhNWYyNWZhZTkyMjI4N2NjNjMzZjIxNzA1ZmUzODcyMzE4MWQ0OWVkZjdlODdjMTE2ZjYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.6hjA7fXueGsedRCutp0iUovwMkyU7LRtj8TWFHmDHmg) ![image](https://private-user-images.githubusercontent.com/13224827/309485657-5340fb15-2a9f-4ed4-9970-227bcf314ba1.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MDk1MjU1MzAsIm5iZiI6MTcwOTUyNTIzMCwicGF0aCI6Ii8xMzIyNDgyNy8zMDk0ODU2NTctNTM0MGZiMTUtMmE5Zi00ZWQ0LTk5NzAtMjI3YmNmMzE0YmExLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzA0VDA0MDcxMFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQ1YWZjNjkzOGZmMGExNDc2MTQ4ZmU2NTNhMTIyMWNiMWExMTJkMjY5NDY0NDc2NGM0ZmY4ZDY4ZTFjMDRmMDImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.4zUNK5PU7sH0UHJ8ty4Aria9D-MRP16b2PhA0JWMnQg) ![image](https://private-user-images.githubusercontent.com/13224827/309485788-9156d2f2-5193-4793-b905-caa861512cef.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MDk1MjU1MzAsIm5iZiI6MTcwOTUyNTIzMCwicGF0aCI6Ii8xMzIyNDgyNy8zMDk0ODU3ODgtOTE1NmQyZjItNTE5My00NzkzLWI5MDUtY2FhODYxNTEyY2VmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzA0VDA0MDcxMFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTZjOGQ2YzUwM2JlZDkyYWJiMjg5ZTNiYmVhMTI4MjY4MjBmNGM4NGQ4MTdhYzQ1Y2ZhZTcwOTM0NTk2N2ZjNzEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.ykSW5Cx38GDB_NLsE1Dvh9GXf5Xj6aRkWQhLpLQOzS8) @pegasas I don't think the `Master` on the ui is a required field, WDYT

@pegasas please Run 'mvn spotless:apply' to fix the code style

> Can we remove the option 'local' from deploy-mode? > > We can only set cluster/client at deploy mode, the local should belong to master config. Maybe we should delete...

> Please update the usage doc. PTAL @ruanwenjun

Hi, @zhongjiajie , Here are the features I'd like to implement in Y2023Q1 :) 1. [Feature] Remote Logging on more targets (Google Cloud Storage, S3, Azure Blob Storage, Etc.) *...

> Search data about operation type @qingwli if user deletes 2 process defintions in a single batch-delete operation, will there be 2 audit records in the table? or just 1...