David,
Thank you for the bug report. You should see notification when it is
addressed.
Rob
On Fri, Oct 12, 2012 at 8:09 PM, David Pinho <david.pinho@...> wrote:
> 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 <david.pinho@...>
> 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
> > <robert.m.manning@...> 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 <david.pinho@...>
> 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
> >>> Squirrel-sql-users@...
> >>> 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
> >> Squirrel-sql-users@...
> >> https://lists.sourceforge.net/lists/listinfo/squirrel-sql-users
> >>
> >
> >
> >
> > --
> > http://davidpinho.net
>
>
>
> --
> 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
> Squirrel-sql-users@...
> https://lists.sourceforge.net/lists/listinfo/squirrel-sql-users
>
|