chore: Cleanup useless 'namespace' in idl/dsn.thrift

apache · May 23, 2024 · d9a6767 · d9a6767
1 parent d00b251
commit d9a6767
Show file tree

Hide file tree

Showing 10 changed files with 9 additions and 5 deletions.
diff --git a/.github/workflows/lint_and_test_admin-cli.yml b/.github/workflows/lint_and_test_admin-cli.yml
@@ -31,6 +31,7 @@ on:
     paths:
       - .github/workflows/lint_and_test_admin-cli.yml
       - admin-cli/**
+      - idl/**
 
   # for manually triggering workflow
   workflow_dispatch:

diff --git a/.github/workflows/lint_and_test_collector.yml b/.github/workflows/lint_and_test_collector.yml
@@ -32,6 +32,7 @@ on:
       - .github/workflows/lint_and_test_collector.yml
       - collector/**
       - go-client/**
+      - idl/**
 
   # for manually triggering workflow
   workflow_dispatch:

diff --git a/.github/workflows/lint_and_test_cpp.yaml b/.github/workflows/lint_and_test_cpp.yaml
@@ -34,6 +34,7 @@ on:
       - docker/thirdparties-src/**
       - CMakeLists.txt
       - cmake_modules/**
+      - idl/**
       - run.sh
       - scripts/pack_server.sh
       - scripts/pack_tools.sh

diff --git a/.github/workflows/lint_and_test_go-client.yml b/.github/workflows/lint_and_test_go-client.yml
@@ -31,6 +31,7 @@ on:
     paths:
       - .github/workflows/lint_and_test_go-client.yml
       - go-client/**
+      - idl/**
 
   # for manually triggering workflow
   workflow_dispatch:

diff --git a/.github/workflows/lint_and_test_java-client.yml b/.github/workflows/lint_and_test_java-client.yml
@@ -26,6 +26,7 @@ on:
         - '*dev'      # developing branch
     paths:
       - .github/workflows/lint_and_test_java-client.yml
+      - idl/**
       - java-client/**
 
   workflow_dispatch:

diff --git a/.github/workflows/lint_and_test_pegic.yml b/.github/workflows/lint_and_test_pegic.yml
@@ -30,6 +30,7 @@ on:
       - '*dev'
     paths:
       - .github/workflows/lint_and_test_pegic.yml
+      - idl/**
       - pegic/**
 
   # for manually triggering workflow

diff --git a/.github/workflows/lint_and_test_scala-client.yml b/.github/workflows/lint_and_test_scala-client.yml
@@ -23,6 +23,7 @@ on:
       - 'v[0-9]+.*' # release branch
     paths:
       - .github/workflows/lint_and_test_scala-client.yml
+      - idl/**
       - scala-client/**
 
   workflow_dispatch:

diff --git a/.github/workflows/test_nodejs-client.yml b/.github/workflows/test_nodejs-client.yml
@@ -25,6 +25,7 @@ on:
         - '*dev'      # developing branch
     paths:
       - .github/workflows/test_nodejs-client.yml
+      - idl/**
       - nodejs-client/**
 
   workflow_dispatch:

diff --git a/.github/workflows/test_python-client.yml b/.github/workflows/test_python-client.yml
@@ -25,6 +25,7 @@ on:
         - '*dev'      # developing branch
     paths:
       - .github/workflows/test_python-client.yml
+      - idl/**
       - python-client/**
 
   workflow_dispatch:

diff --git a/idl/dsn.thrift b/idl/dsn.thrift
@@ -24,11 +24,6 @@
  * THE SOFTWARE.
  */
 
-namespace cpp dsn
-namespace go base
-namespace java org.apache.pegasus.base
-namespace py pypegasus.base
-
 // place holder
 struct rpc_address
 {
-Original file line number
+Diff line change
@@ Expand Up / @@ -30,6 +30,7 @@ on: @@
           - '*dev'
         paths:
           - .github/workflows/lint_and_test_pegic.yml
+          - idl/**
           - pegic/**
       # for manually triggering workflow
@@ Expand Down @@