Skip to content

Conversation

@ny-hub
Copy link
Contributor

@ny-hub ny-hub commented Oct 13, 2025

No description provided.

@ny-hub ny-hub requested a review from allipatev October 13, 2025 13:31
Copy link
Contributor

@allipatev allipatev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left some cosmetic suggestions.


## Problem

Customer is not able to connect to Oracle DB with short host name.Connection can be established with FQDN and IP adrress but not with short host name.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Customer is not able to connect to Oracle DB with short host name.Connection can be established with FQDN and IP adrress but not with short host name.
Customer is not able to connect to Oracle DB with short host name. Connection can be established with FQDN and IP address but not with short host name.

Comment on lines 17 to 21
IMPORT INTO my_table (col1, col2, col4)
FROM ORA
AT my_oracle
USER 'my_user' IDENTIFIED BY 'my_secret'
STATEMENT ' SELECT * FROM orders WHERE order_state=''OK'' '
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
IMPORT INTO my_table (col1, col2, col4)
FROM ORA
AT my_oracle
USER 'my_user' IDENTIFIED BY 'my_secret'
STATEMENT ' SELECT * FROM orders WHERE order_state=''OK'' '
IMPORT INTO my_table (col1, col2, col4)
FROM ORA
AT my_oracle
USER 'my_user' IDENTIFIED BY 'my_secret'
STATEMENT ' SELECT * FROM orders WHERE order_state=''OK'' '

@ny-hub ny-hub requested a review from allipatev October 28, 2025 08:59
@ny-hub ny-hub merged commit f9bf503 into main Oct 29, 2025
1 check passed
@ny-hub ny-hub deleted the ny-hub-patch-1 branch October 29, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants