[GitHub] [inlong-website] EMsnap merged pull request #836: [INLONG-835][Doc] Fix Oracle document errors

2023-08-18 Thread via GitHub


EMsnap merged PR #836:
URL: https://github.com/apache/inlong-website/pull/836


-- 
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-unsubscr...@inlong.apache.org

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



[GitHub] [inlong-website] EMsnap closed issue #835: [Doc] Fix Oracle document errors

2023-08-18 Thread via GitHub


EMsnap closed issue #835: [Doc] Fix Oracle document errors 
URL: https://github.com/apache/inlong-website/issues/835


-- 
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-unsubscr...@inlong.apache.org

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



[inlong-website] branch master updated: [INLONG-835][Doc] Fix Oracle document errors (#836)

2023-08-18 Thread zirui
This is an automated email from the ASF dual-hosted git repository.

zirui pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/inlong-website.git


The following commit(s) were added to refs/heads/master by this push:
 new e5fc78b341 [INLONG-835][Doc] Fix Oracle document errors (#836)
e5fc78b341 is described below

commit e5fc78b341bc5b45b86f6c1febafed6aea17bd82
Author: e-mhui <111486498+e-m...@users.noreply.github.com>
AuthorDate: Fri Aug 18 15:09:16 2023 +0800

[INLONG-835][Doc] Fix Oracle document errors (#836)
---
 docs/data_node/extract_node/oracle-cdc.md | 4 ++--
 .../current/data_node/extract_node/oracle-cdc.md  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/docs/data_node/extract_node/oracle-cdc.md 
b/docs/data_node/extract_node/oracle-cdc.md
index 80609954aa..d4a5dac6a4 100644
--- a/docs/data_node/extract_node/oracle-cdc.md
+++ b/docs/data_node/extract_node/oracle-cdc.md
@@ -220,7 +220,7 @@ Flink SQL> CREATE TABLE oracle_extract_node (
  'schema-name' = 'inlong',
  'table-name' = 'user');
   
--- Read snapshot and binlogs from products table
+-- Read snapshot and redo logs from products table
 Flink SQL> SELECT * FROM oracle_extract_node;
 ```
 **Note:**
@@ -342,7 +342,7 @@ The Oracle Extract Node is a Flink Source connector which 
will read database sna
 
 The config option `scan.startup.mode` specifies the startup mode for Oracle 
CDC consumer. The valid enumerations are:
 
-- `initial` (default): Performs an initial snapshot on the monitored database 
tables upon first startup, and continue to read the latest binlog.
+- `initial` (default): Performs an initial snapshot on the monitored database 
tables upon first startup, and continue to read the latest redo log.
 - `latest-offset`: Never to perform a snapshot on the monitored database 
tables upon first startup, just read from
   the change since the connector was started.
 
diff --git 
a/i18n/zh-CN/docusaurus-plugin-content-docs/current/data_node/extract_node/oracle-cdc.md
 
b/i18n/zh-CN/docusaurus-plugin-content-docs/current/data_node/extract_node/oracle-cdc.md
index e5dd8f0fae..f4400c0219 100644
--- 
a/i18n/zh-CN/docusaurus-plugin-content-docs/current/data_node/extract_node/oracle-cdc.md
+++ 
b/i18n/zh-CN/docusaurus-plugin-content-docs/current/data_node/extract_node/oracle-cdc.md
@@ -345,7 +345,7 @@ Oracle Extract 节点是一个 Flink Source 连接器,它将首先读取数据
 
 配置选项 `scan.startup.mode` 指定 Oracle Extract 节点消费者的启动模式。有效的枚举是:
 
-- `initial` (默认): 首次启动时对被监控的数据库表进行初始快照,并继续读取最新的 Binlog。
+- `initial` (默认): 首次启动时对被监控的数据库表进行初始快照,并继续读取最新的 Redo Log。
 - `latest-offset`: 永远不要在第一次启动时对受监控的数据库表执行快照,只需从自连接器启动以来的更改。
 
 _注意: `scan.startup.mode` 选项的机制依赖于 Debezium 的`snapshot.mode` 
配置。所以请不要一起使用它们。如果您在 DDL 表中同时指定了 `scan.startup.mode` 和 `debezium.snapshot.mode` 
选项,可能会导致 `scan.startup.mode` 不起作用。_



[GitHub] [inlong] sunrisefromdark opened a new pull request, #8767: [INLONG-8767][TubeMQ] Add restart-manager.sh to inlong-tubemq-manager/bin

2023-08-18 Thread via GitHub


sunrisefromdark opened a new pull request, #8767:
URL: https://github.com/apache/inlong/pull/8767

   ### Prepare a Pull Request
   *(Change the title refer to the following example)*
   
   - Title Example: [INLONG-XYZ][Component] Title of the pull request
   
   *(The following *XYZ* should be replaced by the actual [GitHub 
Issue](https://github.com/apache/inlong/issues) number)*
   
   - Fixes #XYZ
   
   ### Motivation
   
   New restart-manager script for fast restart implementation and quick changes 
to the manager configuration.
   
   ### Modifications
   
   **Add restart-manager.sh to inlong-tubemq-manager/bin**
   
   ### Verifying this change
   
   *(Please pick either of the following options)*
   
   - [ ] This change is a trivial rework/code cleanup without any test coverage.
   
   - [ ] This change is already covered by existing tests, such as:
 *(please describe tests)*
   
   - [ ] This change added tests and can be verified as follows:
   
 *(example:)*
 - *Added integration tests for end-to-end deployment with large payloads 
(10MB)*
 - *Extended integration test for recovery after broker failure*
   
   ### Documentation
   
 - Does this pull request introduce a new feature? (yes / no)
 - If yes, how is the feature documented? (not applicable / docs / JavaDocs 
/ not documented)
 - If a feature is not applicable for documentation, explain why?
 - If a feature is not documented yet in this PR, please create a follow-up 
issue for adding the documentation
   


-- 
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-unsubscr...@inlong.apache.org

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



[GitHub] [inlong-website] dockerzhang commented on a diff in pull request #834: [INLONG-834][Doc] Adding more details to the tubemq-manager documentation

2023-08-18 Thread via GitHub


dockerzhang commented on code in PR #834:
URL: https://github.com/apache/inlong-website/pull/834#discussion_r1298220425


##
docs/modules/tubemq/tubemq-manager/quick_start.md:
##
@@ -14,14 +14,16 @@ mysql -uDB_USER -pDB_PASSWD < sql/apache_tube_manager.sql
 
   
 ## Configuration
-- create `tubemanager` and account in MySQL.
-- Add mysql information in conf/application.properties:
-
 ```ini
 # mysql configuration for manager

Review Comment:
   ```suggestion
   # MySQL configuration for Manager
   ```



##
i18n/zh-CN/docusaurus-plugin-content-docs/current/modules/tubemq/tubemq-manager/quick_start.md:
##
@@ -14,14 +14,16 @@ mysql -uDB_USER -pDB_PASSWD < sql/apache_tube_manager.sql
 ```
   
 ## 配置
-- 在mysql中创建`tubemanager`数据和相应用户.
-- 在conf/application.properties中添加mysql信息:
 
 ```ini
 # mysql configuration for manager

Review Comment:
   ```suggestion
   # MySQL configuration for Manager
   ```



-- 
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-unsubscr...@inlong.apache.org

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



[GitHub] [inlong] github-actions[bot] commented on issue #8768: [Feature][TubeMQ] Adding restart-manager.sh to inlong-tubemq-manager/bin

2023-08-18 Thread via GitHub


github-actions[bot] commented on issue #8768:
URL: https://github.com/apache/inlong/issues/8768#issuecomment-1683659374

   Hello @sunrisefromdark, thank you for opening your first issue in InLong 🧡 
We will respond as soon as possible ⏳
   If this is a bug report, please provide screenshots or error logs for us to 
reproduce your issue, so we can do our best to fix it.
   If you have any questions in the meantime, you can also ask us on the 
[InLong Discussions](https://github.com/apache/inlong/discussions) 🔍


-- 
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-unsubscr...@inlong.apache.org

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



[GitHub] [inlong] dockerzhang merged pull request #8756: [INLONG-8755][Dashboard] Data synchronization field mapping optimization

2023-08-18 Thread via GitHub


dockerzhang merged PR #8756:
URL: https://github.com/apache/inlong/pull/8756


-- 
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-unsubscr...@inlong.apache.org

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



[inlong] branch master updated: [INLONG-8755][Dashboard] Data synchronization field mapping optimization (#8756)

2023-08-18 Thread dockerzhang
This is an automated email from the ASF dual-hosted git repository.

dockerzhang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/inlong.git


The following commit(s) were added to refs/heads/master by this push:
 new e1b9598b62 [INLONG-8755][Dashboard] Data synchronization field mapping 
optimization (#8756)
e1b9598b62 is described below

commit e1b9598b6268357149fa61c190ddb32980190f65
Author: Lizhen <88174078+bluew...@users.noreply.github.com>
AuthorDate: Fri Aug 18 22:05:55 2023 +0800

[INLONG-8755][Dashboard] Data synchronization field mapping optimization 
(#8756)

Co-authored-by: Daniel 
---
 .../src/ui/components/FieldList/DetailModal.tsx| 81 +-
 .../src/ui/components/FieldList/index.tsx  |  3 +-
 2 files changed, 35 insertions(+), 49 deletions(-)

diff --git a/inlong-dashboard/src/ui/components/FieldList/DetailModal.tsx 
b/inlong-dashboard/src/ui/components/FieldList/DetailModal.tsx
index c6042aeef4..b8975eb109 100644
--- a/inlong-dashboard/src/ui/components/FieldList/DetailModal.tsx
+++ b/inlong-dashboard/src/ui/components/FieldList/DetailModal.tsx
@@ -17,11 +17,11 @@
  * under the License.
  */
 
-import React, { useEffect, useMemo, useState } from 'react';
-import { Modal, Spin, message, Form, Input, Select } from 'antd';
+import React, { useEffect, useState } from 'react';
+import { Modal, Spin, message, Form, Button } from 'antd';
 import { ModalProps } from 'antd/es/modal';
-import FormGenerator, { useForm } from '@/ui/components/FormGenerator';
-import { useRequest, useUpdateEffect } from '@/ui/hooks';
+import { useForm } from '@/ui/components/FormGenerator';
+import { useRequest } from '@/ui/hooks';
 import { useTranslation } from 'react-i18next';
 import { useDefaultMeta, useLoadMeta, TransformMetaType } from '@/plugins';
 import request from '@/core/utils/request';
@@ -30,22 +30,12 @@ import i18n from '@/i18n';
 import { fieldAllTypes } from './FieldTypeConf';
 
 export interface Props extends ModalProps {
-  // When editing, use the ID to call the interface for obtaining details
-  id?: string;
   inlongGroupId: string;
   inlongStreamId: string;
-  defaultType?: string;
   isSource: boolean;
 }
 
-const Comp: React.FC = ({
-  id,
-  inlongGroupId,
-  inlongStreamId,
-  isSource,
-  defaultType,
-  ...modalProps
-}) => {
+const Comp: React.FC = ({ inlongGroupId, inlongStreamId, isSource, 
...modalProps }) => {
   const [form] = useForm();
   const { t } = useTranslation();
 
@@ -67,7 +57,6 @@ const Comp: React.FC = ({
 }),
 {
   manual: true,
-  formatResult: result => new Entity()?.parse(result) || result,
 },
   );
 
@@ -107,9 +96,6 @@ const Comp: React.FC = ({
   message: i18n.t('meta.Stream.FieldNameRule'),
 },
   ],
-  props: {
-disabled: true,
-  },
 },
 {
   title: i18n.t('meta.Sinks.SourceFieldType'),
@@ -118,7 +104,6 @@ const Comp: React.FC = ({
   initialValue: fieldTypes[0].label,
   rules: [{ required: true }],
   props: {
-disabled: true,
 options: fieldTypes,
   },
 },
@@ -133,9 +118,6 @@ const Comp: React.FC = ({
   message: i18n.t('meta.Stream.FieldNameRule'),
 },
   ],
-  props: (text, record) => ({
-disabled: record.id,
-  }),
 },
 {
   title: i18n.t('meta.Stream.FieldType'),
@@ -143,7 +125,6 @@ const Comp: React.FC = ({
   type: 'select',
   initialValue: '',
   props: (text, record) => ({
-disabled: record.id,
 options: isSource === true ? fieldTypes : fieldAllTypes[sinkType],
   }),
   rules: [{ required: true }],
@@ -166,18 +147,14 @@ const Comp: React.FC = ({
   message: i18n.t('meta.Stream.FieldNameRule'),
 },
   ],
-  props: (text, record) => ({
-disabled: record.id,
-  }),
 },
 {
   title: i18n.t('meta.Stream.FieldType'),
   dataIndex: 'fieldType',
   type: 'select',
   initialValue: '',
-  props: (text, record) => ({
-disabled: record.id,
-options: isSource === true ? fieldTypes : fieldAllTypes[sinkType],
+  props: () => ({
+options: fieldTypes,
   }),
   rules: [{ required: true }],
 },
@@ -196,7 +173,6 @@ const Comp: React.FC = ({
   inlongStreamId,
 };
 if (isUpdate) {
-  submitData.id = id;
   submitData.version = data?.version;
 }
 if (isSource === true) {
@@ -208,11 +184,13 @@ const Comp: React.FC = ({
 },
   });
 } else {
-  sinkData.list[0].sinkFieldList = values.sinkFieldList;
   const sinkSubmitData = {
-...sinkData.list[0],
 inlongGroupId,
 inlongStreamId,
+id: sinkData.list[0].id,
+sinkType: sinkData.list[0].sinkType,
+version: sinkData.list[0].version,
+sinkFieldList: values.sinkFieldList,
   };
   await request({
 url: `/sink/update`,
@@ -227,25 +205,26 @@ co

[GitHub] [inlong] sunrisefromdark closed pull request #8767: [INLONG-8767][TubeMQ] Add restart-manager.sh to inlong-tubemq-manager/bin

2023-08-18 Thread via GitHub


sunrisefromdark closed pull request #8767: [INLONG-8767][TubeMQ] Add 
restart-manager.sh to inlong-tubemq-manager/bin
URL: https://github.com/apache/inlong/pull/8767


-- 
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-unsubscr...@inlong.apache.org

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



[GitHub] [inlong] sunrisefromdark opened a new pull request, #8769: [INLONG-8768][TubeMQ] Add restart-manager.sh to inlong-tubemq-manager…

2023-08-18 Thread via GitHub


sunrisefromdark opened a new pull request, #8769:
URL: https://github.com/apache/inlong/pull/8769

   …/bin
   
   ### Prepare a Pull Request
   *(Change the title refer to the following example)*
   
   - Title Example: [INLONG-XYZ][Component] Title of the pull request
   
   *(The following *XYZ* should be replaced by the actual [GitHub 
Issue](https://github.com/apache/inlong/issues) number)*
   
   - Fixes #8768 
   
   ### Motivation
   
   **New restart-manager script for fast restart implementation and quick 
changes to the manager configuration.**
   
   ### Modifications
   
   **Add restart-manager.sh to inlong-tubemq-manager/bin**
   
   ### Verifying this change
   
   *(Please pick either of the following options)*
   
   - [ ] This change is a trivial rework/code cleanup without any test coverage.
   
   - [ ] This change is already covered by existing tests, such as:
 *(please describe tests)*
   
   - [ ] This change added tests and can be verified as follows:
   
 *(example:)*
 - *Added integration tests for end-to-end deployment with large payloads 
(10MB)*
 - *Extended integration test for recovery after broker failure*
   
   ### Documentation
   
 - Does this pull request introduce a new feature? (yes / no)
 - If yes, how is the feature documented? (not applicable / docs / JavaDocs 
/ not documented)
 - If a feature is not applicable for documentation, explain why?
 - If a feature is not documented yet in this PR, please create a follow-up 
issue for adding the documentation
   


-- 
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-unsubscr...@inlong.apache.org

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



[GitHub] [inlong] H0924cc commented on pull request #8736: [INLONG-8684][Dashboard] Fixed the Vite Server Options (server.fs.deny) can be bypassed using double forward-slash (//)

2023-08-18 Thread via GitHub


H0924cc commented on PR #8736:
URL: https://github.com/apache/inlong/pull/8736#issuecomment-1684317507

   
   
   
   > @H0924cc Can you adjust the lockVersion to v1? too much has changed here.
   
   i try to use"npm install --lockfile-version 1 " hope it work


-- 
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-unsubscr...@inlong.apache.org

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