…nt (apache#1131)

### What changes were proposed in this pull request?
1. add "Manage Metadata Using Gravitino" link to lakehouse iceberg
document
2. merge info blocks

### Why are the changes needed?
more user friendly

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
just docs

…n the markdown (apache#1137)

### What changes were proposed in this pull request?
 Add the `md` suffix for every link in the markdown

### Why are the changes needed?

Fix: apache#1136

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Tested in Docunausars and markdown locally.

---------

Co-authored-by: Heng Qin <qqtt@123.com>

### What changes were proposed in this pull request?

Optimize the docs of Gravitino Trino connector. 

### Why are the changes needed?

Better user experience. 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

N/A

---------

Co-authored-by: yuhui <hui@datastrato.com>

…pache#1146)

### What changes were proposed in this pull request?

Update `./gradlew spotlessApply` of Coding standards in
`Contributing.md`.

### Why are the changes needed?

There is no `grawdlew` file in project, which cause that developer could
not follow coding standards.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

No.

### What changes were proposed in this pull request?

Fix link of REST API page

### Why are the changes needed?

link of REST API page will be changed

### Does this PR introduce _any_ user-facing change?

no

### How was this patch tested?
no need

…en add a column (apache#1145)

### What changes were proposed in this pull request?

Position name supports upper-case when add a column for `FIRST` and
`DEFAULT`.

### Why are the changes needed?

Position name should support upper-case when add a column. For example,
the correct value of position is first NOT FIRST currently.
```
curl -X PUT -H "Accept: application/vnd.gravitino.v1+json" -H "Content-Type: application/json" -d '{
  "updates": [
    {
      "@type": "removeProperty",
      "property": "key2"
    }, {
      "@type": "setProperty",
      "property": "key3",
      "value": "value3"
    },
    {
    "@type": "addColumn",
    "fieldName": [
      "position"
      ],
    "type": "varchar(20)",
    "comment": "Position of user",
    "position": "FIRST"
    } 
  ]
}' http://localhost:8090/api/metalakes/metalake/catalogs/catalog/schemas/schema/tables/table
```

```
Unknown json column position: "FIRST" (through reference chain: com.datastrato.gravitino.dto.requests.TableUpdatesRequest["updates"]->java.util.ArrayList[2])%
```

Fix: apache#1052 

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

`TestTableUpdatesRequest#testAddTableColumnRequest`

…apache#1143)

### What changes were proposed in this pull request?
1. make getCatalogConfig abstract, Hive&Jdbc&Memory implement these
interfaces.
2. remove `gravitino-docker-it` tag from IcebergRESTServiceIT.

### Why are the changes needed?
After apache#711, It takes too much time to test IcebergRESTServiceIT, there's
no need to start the docker container for the memory catalog

Fix: apache#874 

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
existing UT

…inor mistakes in docs (apache#1152)

### What changes were proposed in this pull request?

Refine some descriptions and correct mistakes in Trino-related docs.

### Why are the changes needed?

Make users more aware of the documents.

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

N/A.

…docs (apache#1170)

### What changes were proposed in this pull request?

- Fix typo `Graviton` in `index.md` 
- Fix link mistakes in `index.md` and other sentence problems

### Why are the changes needed?

To Improve the professionalism and rigor of documents 

Fixed apache#1166 

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

N/A.

…s for web to delete node_modules folder (apache#1173)

### What changes were proposed in this pull request?

Optimize clean task of `build.gradle.kts` for web to delete
`node_modules` folder which reduces space after gradle clean.

### Why are the changes needed?

The `node_modules` folder contains libraries downloaded from npm that
takes up a lot of space.

Fix: apache#877

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Test with gradle clean.

…errors in PostgreSQL. (apache#1177)

### What changes were proposed in this pull request?

Fix the issue with joins causing errors in PostgreSQL.
Error message is "Cannot cast
com.datastrato.gravitino.trino.connector.GravitinoColumnHandle to
io.trino.plugin.jdbc.JdbcColumnHandle"

### Why are the changes needed?

Fix: apache#1154

### Does this PR introduce _any_ user-facing change?

NO

### How was this patch tested?

UT

… metalake property (apache#1163)

### What changes were proposed in this pull request?

Remove the `stringId` from metalake property map. 

### Why are the changes needed?

`gravitino.identifier` is generated internally and should not be visible
to users.

Fix: apache#1047 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Existing UTs can cover it.

### What changes were proposed in this pull request?
Fix a typo

### Why are the changes needed?
Just a typo

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?

No need.

Co-authored-by: Heng Qin <qqtt@123.com>

…an be loaded under multiple schemas. (apache#1183)

### What changes were proposed in this pull request?
Fix a bug where the same table name can be loaded under multiple schemas

### Why are the changes needed?
Fix: apache#1171 

### Does this PR introduce _any_ user-facing change?
NO

### How was this patch tested?
UT

Co-authored-by: Clearvive <clearvive@datastrato.com>

… 11, 17 (apache#1199)

### What changes were proposed in this pull request?

This PR proposes to add multiple JDK version support for building and
running Gravitino server, including JDK8, 11, and 17.

Users could specify the version they want to build in
`gradle.properties`, like:

```
jdkVersion = 8

or 

jdkVersion = 11

or 

jdkVersion = 17
```

Or users could specify the version by `./gradlew build -PjdkVersion=8`.

Our generated jars are JDK 8 version compatible, so it can run against
Java runtime 8, 11, and 17.

### Why are the changes needed?

To make Gravitino build, test and run against multiple JDK versions.

Fix: apache#1179 

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

This was compiled, tested, run on JDK8, 11, and 17 locally.

…test sequence dependency (apache#1197)

### What changes were proposed in this pull request?

Remove Junit5 annotation `Order` in test.

### Why are the changes needed?

We need to remove sequence dependencies between test cases so that we
can run them in parallelism.

Fix: apache#995 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Only changes to the test code have been made.

…or drop catalog operation between Trino and Gravitino (apache#1157)

### What changes were proposed in this pull request?

Trino connector will drop catalogs that have been deleted in Gravitino
server.

### Why are the changes needed?

We need to ensure strict synchronization for catalogs between Trino and
Gravitino

Fix: apache#1156 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Add test `testDropCatalogAndCreateAgain` in `TrinoConnectorIT`.

…pache#1194)

### What changes were proposed in this pull request?
 - set `hdfs` as HDFS superuser group in container
- use `datastrato` as the Hive catalog Integration Test user instead of
`root`

### Why are the changes needed?
we should not use the `root` user directly to access HDFS

Fix: apache#554 

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
existing ITs

…he#1205)

### What changes were proposed in this pull request?

This PR proposes to support multiple JDKs in github CI.

### Why are the changes needed?

Fix: apache#1180 

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Existing tests.

…reate to support configurations with no default value (apache#1141)

### What changes were proposed in this pull request?
Some configuration options are necessary but don't have default value.

### Why are the changes needed?

Fix: apache#950 

### Does this PR introduce _any_ user-facing change?
Change some configuration options from `` to none


### How was this patch tested?
CI passed.

---------

Co-authored-by: Heng Qin <qqtt@123.com>

…ty.md` (apache#1214)

### What changes were proposed in this pull request?

Modify the required value for the document `security.md`.
I modify some document mistakes by the way.

### Why are the changes needed?

According the review
apache#1191 (comment)
, modify the required value. If a config option has a default value,
this config option is not required.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
No need.

---------

Co-authored-by: Heng Qin <qqtt@123.com>

…ion (apache#1219)

### What changes were proposed in this pull request?

This is a follow-up work to change the docs to support multiple versions
of JDK.

### Why are the changes needed?

This is a subtask of supporting multiple JDKs for Gravitino.

Fix: apache#1181

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

N/A

…g-gravitino.md (apache#1228)

### What changes were proposed in this pull request?

Fix a low-level error in the document `manage-metadata-using-gravitino`

### Why are the changes needed?

Just fix error. 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

N/A

… ifExisting = true. (apache#1200)

### What changes were proposed in this pull request?
When 'ifExisting' is set to true in the 'deleteColumn' function, it will
validate the existence of the specified column. If the column does not
exist, it will throw an exception.


### Why are the changes needed?
Fix: apache#1112 

### Does this PR introduce _any_ user-facing change?
Users should be aware that when using 'deleteColumn,' it may throw a new
exception, 'IllegalArgumentException.'

### How was this patch tested?
UT

---------

Co-authored-by: Clearvive <clearvive@datastrato.com>

### What changes were proposed in this pull request?
Add the support of custom filters referring to the Spark.

### Why are the changes needed?
Some filters can be configured by users, the needs of users are
different. Some filters are implemented by other libs.
For example, Jetty has implemented the CORS filter, so we supported
custom filters to satisfy the needs of users.

Fix: apache#1190

### Does this PR introduce _any_ user-facing change?

Yes, I have added the document.

### How was this patch tested?
test by the browser console
I test Jetty's cors filter.

Configuration 
```
gravitino.server.webserver.customFilters=org.eclipse.jetty.servlets.CrossOriginFilter
gravitino.server.webserver.org.eclipse.jetty.servlets.CrossOriginFilter.param.allowedOrigins=*
```

Browser console code
```
var http = new XMLHttpRequest();
var url = 'http://localhost:8090/api/version';
http.onreadystatechange = (e) => {
    console.log(http.responseText)
}
http.open("GET", url);
http.send();
```

---------

Co-authored-by: Heng Qin <qqtt@123.com>

…on this address space (apache#1211)

### What changes were proposed in this pull request?

1. Added Docker network overlap detection
2. Reduced IP allocation ranges for lower conflict probability, Modified
to `10.20.30.0/28` only allocate IP ranger `10.20.30.1` ~ `10.20.30.14`
3. Docker network only created on MacOS and default Docker server
environment

### Why are the changes needed?

Fix: apache#879, apache#925

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

CI

…he#1236)

### What changes were proposed in this pull request?

- Add a description about parameter `cascade` when dropping the schema 
- Please provide more details on how to drop the table and clarify
`dropTable` and `purgeTable`

### Why are the changes needed?

Make the document more user-friendly.


### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

N/A

…e ConfigEntry (apache#1233)

### What changes were proposed in this pull request?
Now, ConfigEntry lacks the ability of check config options. So we add
the support of this feature referring to Spark.

### Why are the changes needed?

Fix: apache#1221 

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Add new ut.

---------

Co-authored-by: Heng Qin <qqtt@123.com>

### What changes were proposed in this pull request?
Some collections have O(n) size method, it's a good way to use
`isEmpty`.

### Why are the changes needed?
Improve the quality of code.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

CI passed.

---------

Co-authored-by: Heng Qin <qqtt@123.com>

…yground (apache#1243)

### What changes were proposed in this pull request?

Modify the document according to the pr
apache/gravitino-playground#8
I will submit another pr to modify the document of website.

### Why are the changes needed?

We should keep consistent.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

No.

Co-authored-by: Heng Qin <qqtt@123.com>

### What changes were proposed in this pull request?

- Allow dropping a database when `cascase` is `true`.
- If the database is not empty and `cascade` is `false`, disable the
drop operation.

### Why are the changes needed?

Align the logic with Gravitino EntityStore.

Fix: apache#1231 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

Add UT `testDropMySQLDatabase`

…ssue (apache#1257)

### What changes were proposed in this pull request?

Mysql&Postgres catalog store empty audit info fix.

### Why are the changes needed?
Fix: apache#1253 

### Does this PR introduce _any_ user-facing change?
Users will receive audit information

### How was this patch tested?
IT

---------

Co-authored-by: Clearvive <clearvive@datastrato.com>