-
Notifications
You must be signed in to change notification settings - Fork 58
BUG-49692 #217
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BUG-49692 #217
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 남깁니다.
JDBC User's Manual.md 목차 줄간격도 같이 확인해주세요.
| | Range | [true \| false] | | ||
| | Mandatory | No | | ||
| | Setting Range | The session | | ||
| | Description | Speicifies whether to reuse the ResultSet when executeQuery() is executed from the same PreparedStatement. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
한글 매뉴얼 설명을 보완하면 좋겠습니다.
현재) 같은 PreparedStatement로 부터 executeQuery()를 할때 생성되는 ResultSet을 재사용할 지 여부
제안)
하나의 PreparedStatement 객체에서 executeQuery 메소드를 사용하여 ResultSet객체를 여러 번 생성한 경우 첫 번째 ResultSet 객체를 재사용 할 것인지 설정한다. true는 ResultSet 객체를 재사용한다. 먼저 생성한 ResultSet 객체에 close() 메소드를 사용하면 다음 생성한 ResultSet 객체에 영향을 준다. false는 ResultSet 객체를 재사용하지 않는다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
먼저 생성한 ResultSet 객체에 close() 메소드를 사용하면 다음 생성한 ResultSet 객체에 영향을 준다.
이 부분에서 어떤 영향을 의미하는지 설명해주실 수 있나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
동일한 PreparedStatement 객체에서 executeQuery 메소드를 사용하여 ResultSet 객체를 여러 번 생성한 경우 첫번째 ResultSet 객체를 재사용할 것인지 설정한다. true는 ResultSet 객체를 재사용하므로 먼저 생성한 ResultSet 객체에 close() 메소드를 사용하면 자원이 해제되어 다음 생성한 ResultSet 객체에 메소드 사용 시 오류가 발생한다. 이를 방지하려면 먼저 생성한 ResultSet 객체의 자원을 해제하고 다음 ResultSet 객체를 생성해야 한다. false는 ResultSet 객체를 재사용하지 않는다.
너무 긴가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
약간 변경해봤습니다.
동일한 PreparedStatement 객체에서 executeQuery 메소드를 사용하여 ResultSet 객체를 여러 번 생성한 경우 첫 번째 ResultSet 객체를 재사용할 것인지 설정한다. true는 ResultSet 객체를 재사용한다. 먼저 생성한 ResultSet 객체에 close() 메소드를 사용하면 자원이 해제되어 다른 ResultSet 객체 사용 시 오류가 발생한다. 이를 방지하려면 먼저 생성한 ResultSet 객체의 자원을 해제하고 다음 ResultSet 객체를 생성해야 한다. false는 ResultSet 객체를 재사용하지 않는다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이대로 반영하겠습니다.
| | Range | [true \| false ] | | ||
| | Mandatory | No | | ||
| | Setting Range | N/A | | ||
| | Description | Specifies whether to include function objects in the result of DatabaseMetaData.getProcedures(), DatabaseMetaData.getProcedureColumns(). When this value is set to false, DatabaseMetaData.getFunctions() and DatabaseMetaData.getFunctionColumns() have to be used seperately to retireve the function objects information. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
한글 매뉴얼 설명 보완하면 좋겠습니다.
현재) DatabaseMetaData.getProcedures(), DatabaseMetaData.getProcedureColumns()
의 결과에 function 객체도 포함할지 지정한다. 해당 값을 false로 설정하면 function 객체
정보를 얻기 위해서 DatabaseMetaData.getFunctions()와 DatabaseMetaData.getFunctionColumns()를 별도로 사용해야 한다.
제안)
DatabaseMetaData.getProcedures(), DatabaseMetaData.getProcedureColumns() 메소드 반환 결과에 저장 함수를 포함할지 설정한다.
true는 저장 함수를 포함한다. false는 저장 함수를 포함하지 않는다. 저장 함수 정보를 얻으려면 DatabaseMetaData.getFunctions()와 DatabaseMetaData.getFunctionColumns()를 사용해야 한다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DatabaseMetaData.getProcedures(), DatabaseMetaData.getProcedureColumns() 메소드가 반환하는 결과에 저장 함수를 포함할지 설정한다.
true는 저장 함수를 포함한다. false는 저장 함수를 포함하지 않으므로 저장 함수 정보를 얻으려면 DatabaseMetaData.getFunctions()와 DatabaseMetaData.getFunctionColumns()를 사용해야 한다.
이렇게는 어떠신가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DatabaseMetaData.getProcedures(), DatabaseMetaData.getProcedureColumns() 메소드가 반환하는 결과에 저장 함수를 포함할지 설정한다. true는 저장 함수를 포함한다. false는 저장 함수를 포함하지 않으므로 저장 함수 정보를 얻으려면 DatabaseMetaData.getFunctions()와 DatabaseMetaData.getFunctionColumns()를 사용해야 한다.
이렇게는 어떠신가요?
좋습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해당 내용을 7.1과 trunk에도 반영하려다 보니 7.1에 아래 내용이 작성되어 있고 http://nok.altibase.com/pages/viewpage.action?pageId=66459579 에서도 확인하였습니다. 포함할까요?
원문) JDBC spec. 4.2를 지원하는 Altibase42.jar 에서만 효과가 있으며 Altibase.jar에서는 설정해도 효과가 없다.
수정안) JDBC 4.2를 지원하는 Altibase42.jar에서만 유효한 프로퍼티이다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JDBC spec. 4.2를 지원하는 Altibase42.jar
7.1 에는
JDBC spec. 4.2를 지원하는 Altibase42.jar 와 JDBC spec 3.0을 지원하는 Altibase.jar 를 같이 제공해서 매뉴얼이 다르게 기재된 것입니다.
trunk와 7.2은 불필요한 내용입니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네~ 포함하지 않겠습니다. 감사합니다.
|
|
||
| ##### reuse_resultset | ||
|
|
||
| | Defualt Value | true | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
true 왼쪽 정렬해주세요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정했습니다.
| ##### COUNT_TO_COMMIT | ||
|
|
||
| This specifies the number of changed data(INSERT, DELETE or UPDATE) to be committed at once. The default value is 1000. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
7.1, trunk 영문 매뉴얼에도 빠져있습니다. 같이 반영하면 좋겠습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
반영했습니다.
| | Setting Range | The session | | ||
| | Description | Sets the maximum waiting time for a response. <br/>For more detailed information, please refer to “Timeout” in Chapter 3. | | ||
|
|
||
| ##### reuse_resultset |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
7.1, trunk 영문 매뉴얼에도 추가해주세요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
국문 매뉴얼 업데이트에 맞춰 7.2에 먼저 영문 업데이트하고 피드백받은 후에 추가하겠습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
반영했습니다.
| | Setting Range | N/A | | ||
| | Description | Specifies whether or not to authenticate the server's CA certificate. <br />If this value is false, the client application will not authenticate the server's CA certificate. | | ||
|
|
||
| ##### getprocedures_return_functions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
7.1, trunk 영문 매뉴얼에도 추가해주세요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
국문 매뉴얼 업데이트에 맞춰 7.2에 먼저 영문 업데이트하고 피드백받은 후에 추가하겠습니다.
| | Setting Range | N/A | | ||
| | Description | Specifies whether to include function objects in the result of DatabaseMetaData.getProcedures(), DatabaseMetaData.getProcedureColumns(). When this value is set to false, DatabaseMetaData.getFunctions() and DatabaseMetaData.getFunctionColumns() have to be used seperately to retireve the function objects information. | | ||
|
|
||
| ##### getcolumns_return_jdbctype |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
7.1, trunk 영문 매뉴얼에도 추가해주세요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
반영했습니다.
| | Setting Range | N/A | | ||
| | Description | Specifies the value of DATA_TYPE among the result of DatabaseMetaData.getColumns method. When the value is set to true, it is returned in SQL data type of java.sql.Type specified in JDBC API. When it is set to false, it is returned in data type that is specified in V$DATATYPE. | | ||
|
|
||
| ##### batch_setbytes_use_lob |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
7.1, trunk 영문 매뉴얼에도 추가해주세요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
반영했습니다.
|
@scott-altibase 실장님, 리뷰 부탁드립니다 |
| | 필수 여부 | No | | ||
| | 설정 범위 | N/A | | ||
| | 설명 | DatabaseMetaData.getProcedures(), DatabaseMetaData.getProcedureColumns()<br>의 결과에 function 객체도 포함할지 지정한다. 해당 값을 false로 설정하면 function 객체<br>정보를 얻기 위해서 DatabaseMetaData.getFunctions()와 DatabaseMetaData.getFunctionColumns()를 별도로 사용해야 한다.<br> JDBC spec. 4.2를 지원하는 Altibase42.jar 에서만 효과가 있으며 Altibase.jar에서는 설정해도 효과가 없다.| | ||
| | 설명 | DatabaseMetaData.getProcedures(), DatabaseMetaData.getProcedureColumns() 메소드 반환 결과에 저장 함수를 포함할지 설정한다.<br/>true는 저장 함수를 포함한다. false는 저장 함수를 포함하지 않는다. 저장 함수 정보를 얻으려면 DatabaseMetaData.getFunctions()와 DatabaseMetaData.getFunctionColumns()를 사용해야 한다. JDBC spec. 4.2를 지원하는 Altibase42.jar 에서만 효과가 있으며 Altibase.jar에서는 설정해도 효과가 없다. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
메소드 반환 결과에 저장 함수를 포함할지 설정한다.
-> 메소드 반환 결과에 저장 함수의 포함 여부를 설정한다
| | 필수 여부 | No | | ||
| | 설정 범위 | 세션 | | ||
| | 설명 | 같은 PreparedStatement로 부터 executeQuery()를 할때 생성되는 ResultSet을 재사용할 지 여부 | | ||
| | 설명 | 동일한 PreparedStatement 객체에서 executeQuery() 메소드를 사용하여 ResultSet 객체를 여러 번 생성한 경우 첫 번째 ResultSet 객체를 재사용할 것인지 설정한다. true는 ResultSet 객체를 재사용한다. 먼저 생성한 ResultSet 객체에 close() 메소드를 사용하면 자원이 해제되어 다른 ResultSet 객체 사용 시 오류가 발생한다. 이를 방지하려면 먼저 생성한 ResultSet 객체의 자원을 해제하고 다음 ResultSet 객체를 생성해야 한다. false는 ResultSet 객체를 재사용하지 않는다. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
첫 번째 ResultSet 객체를 재사용할 것인지 설정한다.
-> 첫 번째 ResultSet 객체의 재사용 여부를 설정한다
|
전형진 실장님 의견 괜찮은 것 같은데 반영하고 리뷰 다시 요청해주세요~ |
반영하여 리뷰 re-request 합니다 |
eheejung
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 마치겠습니다. 수고하셨습니다.
scott-altibase
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 마칩니다
|
머지하고 브랜치 삭제합니다. |
7.2 릴리즈 노트 재확인 중 영문 매뉴얼에 누락된 부분이 발견되어 추가합니다.
리뷰 요청드립니다.