|
From: David P. <dav...@gm...> - 2012-10-13 00:09:57
|
So anyway i pulled the source and the following seems to fix it:
### Eclipse Workspace Patch 1.0
#P oracle
Index: src/main/java/net/sourceforge/squirrel_sql/plugins/oracle/tokenizer/OracleQueryTokenizer.java
===================================================================
--- src/main/java/net/sourceforge/squirrel_sql/plugins/oracle/tokenizer/OracleQueryTokenizer.java (revision
6682)
+++ src/main/java/net/sourceforge/squirrel_sql/plugins/oracle/tokenizer/OracleQueryTokenizer.java (working
copy)
@@ -239,7 +239,7 @@
for (Iterator<String> iter = _queries.iterator(); iter.hasNext();)
{
String next = iter.next();
- if (next.startsWith(sep))
+ if (next.startsWith(sep) && !next.startsWith("/*"))
{
tmp.add(sep);
String[] parts = next.split(sep + "\\n+");
Tell me what you think...
On Fri, Oct 12, 2012 at 4:23 PM, David Pinho <dav...@gm...> wrote:
> Done.
>
> Added the stacktrace from SQuirreL logs:
>
> 2012-10-12 16:17:28,429 [Thread-5] ERROR
> net.sourceforge.squirrel_sql.client.session.MessagePanel -
> privateShowMessage: Exception was Warning: execution completed with
> warning
> java.sql.SQLWarning: Warning: execution completed with warning
> at oracle.jdbc.driver.DatabaseError.addSqlWarning(DatabaseError.java:945)
> at oracle.jdbc.driver.DatabaseError.addSqlWarning(DatabaseError.java:993)
> at oracle.jdbc.driver.DatabaseError.addSqlWarning(DatabaseError.java:1008)
> at oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:220)
> at oracle.jdbc.driver.T4CStatement.executeForRows(T4CStatement.java:1033)
> at oracle.jdbc.driver.OracleStatement.doExecuteWithTimeout(OracleStatement.java:1329)
> at oracle.jdbc.driver.OracleStatement.executeInternal(OracleStatement.java:1909)
> at oracle.jdbc.driver.OracleStatement.execute(OracleStatement.java:1871)
> at oracle.jdbc.driver.OracleStatementWrapper.execute(OracleStatementWrapper.java:318)
> at net.sourceforge.squirrel_sql.client.session.SQLExecuterTask.processQuery(SQLExecuterTask.java:410)
> at net.sourceforge.squirrel_sql.client.session.SQLExecuterTask.run(SQLExecuterTask.java:220)
> at net.sourceforge.squirrel_sql.fw.util.TaskExecuter.run(TaskExecuter.java:82)
> at java.lang.Thread.run(Unknown Source)
>
>
> On Fri, Oct 12, 2012 at 12:11 PM, Robert Manning
> <rob...@gm...> wrote:
>> That would be a bug in the Oracle plugin's query tokenizer. Can you put in
>> a bug ticket for that here:
>>
>> http://sourceforge.net/tracker/?group_id=28383&atid=393414
>>
>> Rob
>>
>> On Fri, Oct 12, 2012 at 4:42 AM, David Pinho <dav...@gm...> wrote:
>>>
>>> Hi all,
>>>
>>> I'm seeing a strange behavior (well strange for me anyway). When
>>> creating a package in an Oracle DB:
>>>
>>> If i execute this in SQuirreL:
>>>
>>> CREATE OR REPLACE PACKAGE foo AS
>>>
>>> /* comment */
>>> c1 CONSTANT INTEGER := 1;
>>>
>>> FUNCTION f1 (arg1 IN NUMBER, arg2 IN DATE) RETURN INTEGER;
>>>
>>> END foo;
>>> /
>>>
>>> It will execute fine and create the package. But if i change the
>>> position of the comment line:
>>>
>>> CREATE OR REPLACE PACKAGE foo AS
>>>
>>> c1 CONSTANT INTEGER := 1;
>>> /* comment */
>>>
>>> FUNCTION f1 (arg1 IN NUMBER, arg2 IN DATE) RETURN INTEGER;
>>>
>>> END foo;
>>> /
>>>
>>> it will fail with the error:
>>> Query 1 of 4, Rows read: 0, Elapsed time (seconds) - Total: 0.02, SQL
>>> query: 0.02, Reading results: 0
>>> Error: Invalid SQL type: sqlKind = 0
>>> SQLState: 99999
>>> ErrorCode: 17439
>>> Position: 0
>>> Error occured in:
>>> /* comment *
>>>
>>>
>>> If i run it in SQLPlus it will run just fine. Any ideias what this
>>> might be or how i can get around it?
>>>
>>> I'm using Oracle 10.2.0.4.0 and SQuirreL 3.4.0 (Oracle Thin Driver).
>>>
>>>
>>> Tks,
>>>
>>> David
>>>
>>>
>>>
>>>
>>> --
>>> http://davidpinho.net
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> Don't let slow site performance ruin your business. Deploy New Relic APM
>>> Deploy New Relic app performance management and know exactly
>>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>>> http://p.sf.net/sfu/newrelic-dev2dev
>>> _______________________________________________
>>> Squirrel-sql-users mailing list
>>> Squ...@li...
>>> https://lists.sourceforge.net/lists/listinfo/squirrel-sql-users
>>
>>
>>
>> ------------------------------------------------------------------------------
>> Don't let slow site performance ruin your business. Deploy New Relic APM
>> Deploy New Relic app performance management and know exactly
>> what is happening inside your Ruby, Python, PHP, Java, and .NET app
>> Try New Relic at no cost today and get our sweet Data Nerd shirt too!
>> http://p.sf.net/sfu/newrelic-dev2dev
>> _______________________________________________
>> Squirrel-sql-users mailing list
>> Squ...@li...
>> https://lists.sourceforge.net/lists/listinfo/squirrel-sql-users
>>
>
>
>
> --
> http://davidpinho.net
--
http://davidpinho.net
|
